[oe] vlc_1.1.4.1: Why is `lua5.1-native` not added to `DEPENDS` automatically? (was: [PATCH] vlc_1.1.4.1: add `lua5.1` to `DEPENDS`)

[Paul Menzel]

Am Mittwoch, den 03.11.2010, 10:55 -0700 schrieb Dallas Foley:
> On 10-11-03 02:28 AM, Paul Menzel wrote:
> > Date: Wed, 3 Nov 2010 09:52:36 +0100
> >
> > After VLC 1.0.6, being the latest version in OE until addition of 1.1.4.1 in commit 30e362 [2], some interfaces were implemented in Lua in VLC 1.1.0 [1].
> >
> > 	Interfaces:
> > 	 * Renamed the legacy rc, telnet and http interfaces to oldrc, oldtelnet and oldhttp.
> > 	 * rc, telnet and http are now implemented using the lua interface system.
> >
> > Task `configure` fails with the following error message.
> >
> > 	[…]
> > 	| checking for LUA... no
> > 	| configure: WARNING: lua5.1 not found, trying lua>= 5.1 instead
> > 	| checking for LUA... no
> > 	| checking lua.h usability... no
> > 	| checking lua.h presence... no
> > 	| checking for lua.h... no
> > 	| checking lauxlib.h usability... no
> > 	| checking lauxlib.h presence... no
> > 	| checking for lauxlib.h... no
> > 	| checking lualib.h usability... no
> > 	| checking lualib.h presence... no
> > 	| checking for lualib.h... no
> > 	| checking for luaL_newstate in -llua5.1 ... no
> > 	| checking for luaL_newstate in -llua51 ... no
> > 	| checking for luaL_newstate in -llua ... no
> > 	| configure: error: Could not find lua. Lua is needed for some interfaces (rc, telnet, http) as well as many other custom scripts. Use --disable-lua to ignore this error.
> > 	| ERROR: Function do_configure failed
> > 	NOTE: package vlc-1.1.4.1-r0: task do_configure: Failed
> > 	[…]
> >
> > Adding `lua5.1` to `DEPENDS` is chosen in favor of disabling it.
> >
> > [1] http://git.videolan.org/?p=vlc.git;a=commitdiff;h=3c1df96cda8086f605f2eacaa5653c9e43ec45ac
> > [2] http://cgit.openembedded.org/cgit.cgi/openembedded/commit/?id=30e362c22a49521ebeef9bed1f0c58902b7dc50b
> >
> > Signed-off-by: Paul Menzel<paulepanter at users.sourceforge.net>
> > ---
> >   recipes/vlc/vlc_1.1.4.1.bb |    2 +-
> >   1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/recipes/vlc/vlc_1.1.4.1.bb b/recipes/vlc/vlc_1.1.4.1.bb
> > index d4535a5..ef764c1 100644
> > --- a/recipes/vlc/vlc_1.1.4.1.bb
> > +++ b/recipes/vlc/vlc_1.1.4.1.bb
> > @@ -10,7 +10,7 @@ SRC_URI[sha256sum] = "61c9ea30a17ea40c6ccbfd507026e5c83ad9e0691f221d3667c8e49696
> >
> >   # ffmpeg from git (library version =>  52) is required
> >   # libtool-native must be>= 2.2.4
> > -DEPENDS += "libdvdcss libdvdread"
> > +DEPENDS += "libdvdcss libdvdread lua"
> >
> >   EXTRA_OECONF += "\
> >   	--enable-dvdread \
>
> I seem to need lua5.1 and lua5.1-native in DEPENDS for this vlc as opposed to just lua.

You are indeed correct. Somehow I messed up the patch and just edited
the commit message. :( I sent a fixup some minutes ago [3].

> ie. DEPENDS += "libdvdcss libdvdread lua5.1 lua5.1-native"
> 
> I doubt this has anything to do with it, but I am using DISTRO="angstrom-2010.x"
> and setting
> ANGSTROM_LIBTOOL_VERSION = "2.2.6b"
> LIBTOOL_HAS_SYSROOT = "no"
> only because apache2 does not configure with the newer libtool

As you wrote after applying this patch there is still the following
problem in the task `configure`.

        […]
        checking for LUA... yes
        checking for luac... no
        configure: error: Could not find the LUA byte compiler.
        ERROR: Function do_configure failed

Indeed in all my tests there is no `luac` staged.

        $ find . -name *luac*
        ./minimal-dev/sysroots/armv7a-oe-linux-gnueabi/usr/include/luaconf.h
        ./minimal-dev/sysroots/armv7a-oe-linux-gnueabi/usr/share/man/man1/luac.1
        ./minimal-dev/work/armv7a-oe-linux-gnueabi/vlc-1.1.4.1-r0/vlc-1.1.4.1/share/lua/intf/luac.lua
        $ find angstrom-dev/ -name *luac* # angstrom-2008.1
        angstrom-dev/sysroots/armv7a-angstrom-linux-gnueabi/usr/include/luaconf.h
        angstrom-dev/sysroots/armv7a-angstrom-linux-gnueabi/usr/share/man/man1/luac.1
        angstrom-dev/work/armv7a-angstrom-linux-gnueabi/vlc-1.1.4.1-r0/vlc-1.1.4.1/share/lua/intf/luac.lua

This should be done by adding `lua5.1-native` to `DEPENDS` as you
suggested. But I am wondering why `lua5.1` is not providing `luac`. Is
there a problem with the `lua5.1` recipe [4][5]?


Thanks,

Paul


[3] http://patchwork.openembedded.org/patch/3504/
[4] http://cgit.openembedded.org/cgit.cgi/openembedded/tree/recipes/lua/lua5.1_5.1.4.bb?id=48838f0fca6b00767115dd13bce2537f035fd3ba
[5] http://cgit.openembedded.org/cgit.cgi/openembedded/tree/recipes/lua/lua.inc?id=709c4d66e0b107ca606941b988bad717c0b45d9b
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20101106/d08b2eb8/attachment-0002.sig>