[oe] [meta-oe][PATCH v3] luajit: Upgrade to 2.1.0-beta3

[Khem Raj]

On 2/24/20 11:38 PM, Leo Yan wrote:
> Hi Khem,
> 
> On Mon, Feb 24, 2020 at 10:41:04PM -0800, Khem Raj wrote:
>>
>>
>> On 2/24/20 10:37 PM, Leo Yan wrote:
>>> Since luajit 2.1.0-beta3 can support architecture aarch64 and the old
>>> misses to support aarch64, the patch upgrades to luajit 2.1.0-beta3.
>>>
>>> Signed-off-by: Leo Yan <leo.yan at linaro.org>
>>> ---
>>>    .../luajit/luajit/clang.patch                 | 19 -------------------
>>
>>
>> why is this patch dropped ? it building with clang handled differently now ?
>> if so please add a line or two in commit msg
> 
> To be honest, I don't know how to hit the condition for
> '#elif !LJ_TARGET_PS3', and since considering Clang has upgraded to new
> version (9.0.1), this is another reason I dropped this patch.
> 

you could easily excercise the need for it, by adding meta-clang and setting

TOOLCHAIN_pn-luajit = "clang" (in local.conf)
bitbake luajit

> But I cannot give out any concrete reason, will keep this patch.
> 
> Thanks,
> Leo
> 
>>>    .../luajit/{luajit_2.0.5.bb => luajit_git.bb} | 14 ++++++--------
>>>    2 files changed, 6 insertions(+), 27 deletions(-)
>>>    delete mode 100644 meta-oe/recipes-devtools/luajit/luajit/clang.patch
>>>    rename meta-oe/recipes-devtools/luajit/{luajit_2.0.5.bb => luajit_git.bb} (89%)
>>>
>>> diff --git a/meta-oe/recipes-devtools/luajit/luajit/clang.patch b/meta-oe/recipes-devtools/luajit/luajit/clang.patch
>>> deleted file mode 100644
>>> index c39ef6fd4..000000000
>>> --- a/meta-oe/recipes-devtools/luajit/luajit/clang.patch
>>> +++ /dev/null
>>> @@ -1,19 +0,0 @@
>>> -clang pretends to be gcc 4.2.0 which is a big lie when it comes
>>> -to features, its same as latest gcc
>>> -
>>> -Signed-off-by: Khem Raj <raj.khem at gmail.com>
>>> -Upstream-Status: Pending
>>> -
>>> -Index: LuaJIT-2.0.5/src/lj_arch.h
>>> -===================================================================
>>> ---- LuaJIT-2.0.5.orig/src/lj_arch.h
>>> -+++ LuaJIT-2.0.5/src/lj_arch.h
>>> -@@ -313,7 +313,7 @@
>>> - #error "Need at least GCC 4.2 or newer"
>>> - #endif
>>> - #elif !LJ_TARGET_PS3
>>> --#if (__GNUC__ < 4) || ((__GNUC__ == 4) && __GNUC_MINOR__ < 3)
>>> -+#if (__GNUC__ < 4) || ((__GNUC__ == 4) && __GNUC_MINOR__ < 3) && !defined(__clang__)
>>> - #error "Need at least GCC 4.3 or newer"
>>> - #endif
>>> - #endif
>>> diff --git a/meta-oe/recipes-devtools/luajit/luajit_2.0.5.bb b/meta-oe/recipes-devtools/luajit/luajit_git.bb
>>> similarity index 89%
>>> rename from meta-oe/recipes-devtools/luajit/luajit_2.0.5.bb
>>> rename to meta-oe/recipes-devtools/luajit/luajit_git.bb
>>> index 93128dda8..da798328e 100644
>>> --- a/meta-oe/recipes-devtools/luajit/luajit_2.0.5.bb
>>> +++ b/meta-oe/recipes-devtools/luajit/luajit_git.bb
>>> @@ -1,14 +1,13 @@
>>>    SUMMARY = "Just-In-Time Compiler for Lua"
>>>    LICENSE = "MIT"
>>> -LIC_FILES_CHKSUM = "file://COPYRIGHT;md5=10a96c93403affcc34765f4c2612bc22"
>>> +LIC_FILES_CHKSUM = "file://COPYRIGHT;md5=d739bb9250a55c124a545b588fd76771"
>>>    HOMEPAGE = "http://luajit.org"
>>> -PV .= "+git${SRCPV}"
>>> -SRCREV = "02b521981a1ab919ff2cd4d9bcaee80baf77dce2"
>>> -SRC_URI = "git://luajit.org/git/luajit-2.0.git;protocol=http \
>>> +PV = "2.0.5+2.1.0-beta3"
>>> +SRCREV = "0ad60ccbc3768fa8e3e726858adf261950edbc22"
>>> +SRC_URI = "git://luajit.org/git/luajit-2.0.git;protocol=http;branch=v2.1 \
>>>               file://0001-Do-not-strip-automatically-this-leaves-the-stripping.patch \
>>> -           file://clang.patch \
>>> -"
>>> +           "
>>>    S = "${WORKDIR}/git"
>>> @@ -90,8 +89,7 @@ FILES_${PN}-dev += "${libdir}/libluajit-5.1.a \
>>>    "
>>>    FILES_luajit-common = "${datadir}/${BPN}-${PV}"
>>> -# Aarch64/mips64/ppc/ppc64/riscv64 is not supported in this release
>>> -COMPATIBLE_HOST_aarch64 = "null"
>>> +# mips64/ppc/ppc64/riscv64 is not supported in this release
>>>    COMPATIBLE_HOST_mipsarchn32 = "null"
>>>    COMPATIBLE_HOST_mipsarchn64 = "null"
>>>    COMPATIBLE_HOST_powerpc = "null"
>>>