[oe] [PATCH][meta-oe] Revert "luajit: Upgrade to 2.1 beta1"

Khem Raj raj.khem at gmail.com
Mon Nov 9 17:50:39 UTC 2015


On Mon, Nov 9, 2015 at 1:43 AM, Martin Jansa <martin.jansa at gmail.com> wrote:
> On Sun, Nov 08, 2015 at 07:41:28PM -0800, Khem Raj wrote:
>> Martin
>>
>> The build failures are not due to luajit, they are due to lua itself
>> being upgraded beyond 5.1 and its ABI incompatible. Its being
>> highlighted
>> by luajit 2.1 most probbably, If the lua 5.1 reintroduction patch has
>> been applied then we can look into the fact that lua 5.1 is chosen for
>> these packages.
>
> It's failing with this commit even without lua 5.2 upgrade.
>
> And it's beta anyway, so I would like to revert this for jethro branch
> and then it can be re-applied in master when sysdig and devilspie are
> working correctly with it, ok?

for jethro thats fine, let me see this week if I can come up with a
fix I was not getting the compile issues in my env so obviously I am
missing something.

>
>>
>> and yes the 32bit host dependencies has always been a requirement for luajit.
>>
>> On Sun, Nov 8, 2015 at 12:50 PM, Martin Jansa <martin.jansa at gmail.com> wrote:
>> > Was it needed for 2.0 as well? If yes, then I'll re-add it.
>> >
>> > On Sun, Nov 8, 2015 at 9:32 PM, Andreas Müller <schnitzeltony at googlemail.com
>> >> wrote:
>> >
>> >> On Sun, Nov 8, 2015 at 12:07 PM, Martin Jansa <martin.jansa at gmail.com>
>> >> wrote:
>> >> > This reverts commit 2ce3086aa859178fb5f7d6db9876229c8ed5a6eb.
>> >> >
>> >> > Causes sysdig and devilspie2 build failures.
>> >> >
>> >> > Signed-off-by: Martin Jansa <Martin.Jansa at gmail.com>
>> >> > ---
>> >> >  ...p-automatically-this-leaves-the-stripping.patch | 33 +++++++++++
>> >> >  meta-oe/recipes-devtools/luajit/luajit_2.0.4.bb    | 56
>> >> ++++++++++++++++++
>> >> >  meta-oe/recipes-devtools/luajit/luajit_2.1.bb      | 67
>> >> ----------------------
>> >> >  3 files changed, 89 insertions(+), 67 deletions(-)
>> >> >  create mode 100644
>> >> meta-oe/recipes-devtools/luajit/luajit/0001-Do-not-strip-automatically-this-leaves-the-stripping.patch
>> >> >  create mode 100644 meta-oe/recipes-devtools/luajit/luajit_2.0.4.bb
>> >> >  delete mode 100644 meta-oe/recipes-devtools/luajit/luajit_2.1.bb
>> >> >
>> >> > diff --git
>> >> a/meta-oe/recipes-devtools/luajit/luajit/0001-Do-not-strip-automatically-this-leaves-the-stripping.patch
>> >> b/meta-oe/recipes-devtools/luajit/luajit/0001-Do-not-strip-automatically-this-leaves-the-stripping.patch
>> >> > new file mode 100644
>> >> > index 0000000..f123b3f
>> >> > --- /dev/null
>> >> > +++
>> >> b/meta-oe/recipes-devtools/luajit/luajit/0001-Do-not-strip-automatically-this-leaves-the-stripping.patch
>> >> > @@ -0,0 +1,33 @@
>> >> > +From 528009f5ca0685c95ef27f6c8957160499c0576a Mon Sep 17 00:00:00 2001
>> >> > +From: Khem Raj <raj.khem at gmail.com>
>> >> > +Date: Fri, 10 May 2013 10:42:45 -0700
>> >> > +Subject: [PATCH] Do not strip automatically, this leaves the stripping
>> >> > + busines to OE
>> >> > +
>> >> > +Upstream-Status: Inappropriate [OE-Specific]
>> >> > +
>> >> > +Signed-off-by: Khem Raj <raj.khem at gmail.com>
>> >> > +---
>> >> > + src/Makefile |    2 --
>> >> > + 1 file changed, 2 deletions(-)
>> >> > +
>> >> > +diff --git a/src/Makefile b/src/Makefile
>> >> > +index 278324a..5ecd6d5 100644
>> >> > +--- a/src/Makefile
>> >> > ++++ b/src/Makefile
>> >> > +@@ -665,12 +665,10 @@ $(LUAJIT_A): $(LJVMCORE_O)
>> >> > + $(LUAJIT_SO): $(LJVMCORE_O)
>> >> > +       $(E) "DYNLINK   $@"
>> >> > +       $(Q)$(TARGET_LD) $(TARGET_ASHLDFLAGS) -o $@ $(LJVMCORE_DYNO)
>> >> $(TARGET_ALIBS)
>> >> > +-      $(Q)$(TARGET_STRIP) $@
>> >> > +
>> >> > + $(LUAJIT_T): $(TARGET_O) $(LUAJIT_O) $(TARGET_DEP)
>> >> > +       $(E) "LINK      $@"
>> >> > +       $(Q)$(TARGET_LD) $(TARGET_ALDFLAGS) -o $@ $(LUAJIT_O)
>> >> $(TARGET_O) $(TARGET_ALIBS)
>> >> > +-      $(Q)$(TARGET_STRIP) $@
>> >> > +       $(E) "OK        Successfully built LuaJIT"
>> >> > +
>> >> > +
>> >> ##############################################################################
>> >> > +--
>> >> > +1.7.9.5
>> >> > +
>> >> > diff --git a/meta-oe/recipes-devtools/luajit/luajit_2.0.4.bb
>> >> b/meta-oe/recipes-devtools/luajit/luajit_2.0.4.bb
>> >> > new file mode 100644
>> >> > index 0000000..4a15a4e
>> >> > --- /dev/null
>> >> > +++ b/meta-oe/recipes-devtools/luajit/luajit_2.0.4.bb
>> >> > @@ -0,0 +1,56 @@
>> >> > +SUMMARY = "Just-In-Time Compiler for Lua"
>> >> > +LICENSE = "MIT"
>> >> > +LIC_FILES_CHKSUM =
>> >> "file://COPYRIGHT;md5=3992f1fbae3b8b061f9056b7fcda8cc6"
>> >> > +HOMEPAGE = "http://luajit.org"
>> >> > +
>> >> > +SRC_URI = "http://luajit.org/download/LuaJIT-${PV}.tar.gz \
>> >> > +
>> >>  file://0001-Do-not-strip-automatically-this-leaves-the-stripping.patch \
>> >> > +"
>> >> > +SRC_URI[md5sum] = "dd9c38307f2223a504cbfb96e477eca0"
>> >> > +SRC_URI[sha256sum] =
>> >> "620fa4eb12375021bef6e4f237cbd2dd5d49e56beb414bee052c746beef1807d"
>> >> > +
>> >> > +S = "${WORKDIR}/LuaJIT-${PV}"
>> >> > +
>> >> > +inherit pkgconfig binconfig
>> >> > +
>> >> > +BBCLASSEXTEND = "native"
>> >> > +
>> >> > +do_configure_prepend() {
>> >> > +    sed -i 's:PREFIX= /usr/local:PREFIX= ${prefix}:g' ${S}/Makefile
>> >> > +    sed -i 's:MULTILIB= lib:MULTILIB= ${baselib}:g' ${S}/Makefile
>> >> > +}
>> >> > +
>> >> > +EXTRA_OEMAKE = 'CROSS=${HOST_PREFIX} \
>> >> > +                TARGET_CFLAGS="${TOOLCHAIN_OPTIONS} ${HOST_CC_ARCH}" \
>> >> > +                TARGET_LDFLAGS="${TOOLCHAIN_OPTIONS}" \
>> >> > +                TARGET_SHLDFLAGS="${TOOLCHAIN_OPTIONS}"'
>> >> > +EXTRA_OEMAKE_append_powerpc = ' HOST_CC="${BUILD_CC} -m32"'
>> >> > +EXTRA_OEMAKE_append_x86 = ' HOST_CC="${BUILD_CC} -m32"'
>> >> > +EXTRA_OEMAKE_append_x86-64 = ' HOST_CC="${BUILD_CC}"'
>> >> > +EXTRA_OEMAKE_append_powerpc64 = ' HOST_CC="${BUILD_CC}"'
>> >> > +EXTRA_OEMAKE_append_arm = ' HOST_CC="${BUILD_CC} -m32"'
>> >> > +EXTRA_OEMAKE_append_mips64 = ' HOST_CC="${BUILD_CC} -m32"'
>> >> > +
>> >> > +do_compile () {
>> >> > +    oe_runmake
>> >> > +}
>> >> > +
>> >> > +do_install () {
>> >> > +    oe_runmake 'DESTDIR=${D}' install
>> >> > +    rmdir ${D}${datadir}/lua/5.* \
>> >> > +          ${D}${datadir}/lua \
>> >> > +          ${D}${libdir}/lua/5.* \
>> >> > +          ${D}${libdir}/lua
>> >> > +}
>> >> > +
>> >> > +PACKAGES += 'luajit-common'
>> >> > +
>> >> > +FILES_${PN} += "${libdir}/libluajit-5.1.so.2 \
>> >> > +    ${libdir}/libluajit-5.1.so.${PV} \
>> >> > +"
>> >> > +FILES_${PN}-dev += "${libdir}/libluajit-5.1.a \
>> >> > +    ${libdir}/libluajit-5.1.so \
>> >> > +    ${libdir}/pkgconfig/luajit.pc \
>> >> > +"
>> >> > +FILES_luajit-common = "${datadir}/${BPN}-${PV}"
>> >> > +
>> >> > diff --git a/meta-oe/recipes-devtools/luajit/luajit_2.1.bb
>> >> b/meta-oe/recipes-devtools/luajit/luajit_2.1.bb
>> >> > deleted file mode 100644
>> >> > index 5cee532..0000000
>> >> > --- a/meta-oe/recipes-devtools/luajit/luajit_2.1.bb
>> >> > +++ /dev/null
>> >> > @@ -1,67 +0,0 @@
>> >> > -SUMMARY = "Just-In-Time Compiler for Lua"
>> >> > -LICENSE = "MIT"
>> >> > -LIC_FILES_CHKSUM =
>> >> "file://COPYRIGHT;md5=3992f1fbae3b8b061f9056b7fcda8cc6"
>> >> > -HOMEPAGE = "http://luajit.org"
>> >> > -
>> >> > -SRCREV = "776c6935ed807fc3b1ee6cd89a0cf682ffb7e9de"
>> >> > -SRC_URI = "git://
>> >> luajit.org/git/luajit-2.0.git;protocol=http;branch=v${PV} \
>> >> > -"
>> >> > -
>> >> > -SPIN = ".0-beta1"
>> >> > -
>> >> > -S = "${WORKDIR}/git"
>> >> > -
>> >> > -inherit pkgconfig binconfig
>> >> > -
>> >> > -BBCLASSEXTEND = "native"
>> >> > -
>> >> > -do_configure_prepend() {
>> >> > -    sed -i 's:PREFIX= /usr/local:PREFIX= ${prefix}:g' ${S}/Makefile
>> >> > -    sed -i 's:MULTILIB= lib:MULTILIB= ${baselib}:g' ${S}/Makefile
>> >> > -}
>> >> > -
>> >> > -EXTRA_OEMAKE = 'CROSS=${HOST_PREFIX} \
>> >> > -                TARGET_CFLAGS="${TOOLCHAIN_OPTIONS} ${HOST_CC_ARCH}" \
>> >> > -                TARGET_LDFLAGS="${TOOLCHAIN_OPTIONS}" \
>> >> > -                TARGET_SHLDFLAGS="${TOOLCHAIN_OPTIONS}"
>> >> TARGET_STRIP="/bin/echo"'
>> >> > -
>> >> Would it be a big issue to re-add comment below - I was not aware of
>> >> the build host requirements and there could be other users
>> >> > -# Important note: On the build machine bit compatible glibc-dev and
>> >> libgcc must
>> >> > -# be installed. E.g if the target architecture has 32 bit (see '-m32'
>> >> below),
>> >> > -# 32 bit versions of glibc-dev/libgcc (e.g glibc-devel.i686 /
>> >> libgcc.i686 on
>> >> > -# fedora) are required.
>> >> > -#
>> >> > -# Otherwise the following error occurs:
>> >> > -# | /usr/include/gnu/stubs.h:7:27: fatal error: gnu/stubs-32.h: No such
>> >> file or directory
>> >> > -# |  # include <gnu/stubs-32.h>
>> >> > -
>> >> Andreas
>> >> --
>> >> _______________________________________________
>> >> Openembedded-devel mailing list
>> >> Openembedded-devel at lists.openembedded.org
>> >> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>> >>
>> > --
>> > _______________________________________________
>> > Openembedded-devel mailing list
>> > Openembedded-devel at lists.openembedded.org
>> > http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>> --
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel at lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>
> --
> Martin 'JaMa' Jansa     jabber: Martin.Jansa at gmail.com
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>



More information about the Openembedded-devel mailing list