All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] vlc_1.1.4.1: add `lua5.1` to `DEPENDS`
@ 2010-11-03  9:28 Paul Menzel
  2010-11-03 17:55 ` Dallas Foley
  0 siblings, 1 reply; 12+ messages in thread
From: Paul Menzel @ 2010-11-03  9:28 UTC (permalink / raw)
  To: openembedded-devel

[-- Attachment #1: Type: text/plain, Size: 2245 bytes --]

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@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 \
-- 
1.7.2.3

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [PATCH] vlc_1.1.4.1: add `lua5.1` to `DEPENDS`
  2010-11-03  9:28 [PATCH] vlc_1.1.4.1: add `lua5.1` to `DEPENDS` Paul Menzel
@ 2010-11-03 17:55 ` Dallas Foley
  2010-11-03 20:08   ` Khem Raj
                     ` (2 more replies)
  0 siblings, 3 replies; 12+ messages in thread
From: Dallas Foley @ 2010-11-03 17:55 UTC (permalink / raw)
  To: openembedded-devel; +Cc: Paul Menzel

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@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 \
>
>
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel

I seem to need lua5.1 and lua5.1-native in DEPENDS for this vlc as opposed to just lua.
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








^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] vlc_1.1.4.1: add `lua5.1` to `DEPENDS`
  2010-11-03 17:55 ` Dallas Foley
@ 2010-11-03 20:08   ` Khem Raj
  2010-11-05 22:55   ` [PATCH] vlc_1.1.4.1: Really add `lua5.1` to `DEPENDS` (instead of just `lua`) Paul Menzel
  2010-11-05 23:21   ` 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
  2 siblings, 0 replies; 12+ messages in thread
From: Khem Raj @ 2010-11-03 20:08 UTC (permalink / raw)
  To: openembedded-devel; +Cc: Paul Menzel

On Wed, Nov 3, 2010 at 10:55 AM, Dallas Foley <dfoley@telus.net> wrote:
> 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@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 \
>>
>>
>>
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel@lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>
> I seem to need lua5.1 and lua5.1-native in DEPENDS for this vlc as opposed
> to just lua.
> 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
>

did you report the apache problem and details about it before ?
if not the please do. I would say we should find the reasoning when we
add something to DEPENDS
the addition is simple but impact may be too far reaching.

>
>
>
>
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>



^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH] vlc_1.1.4.1: Really add `lua5.1` to `DEPENDS` (instead of just `lua`).
  2010-11-03 17:55 ` Dallas Foley
  2010-11-03 20:08   ` Khem Raj
@ 2010-11-05 22:55   ` Paul Menzel
  2010-11-05 23:21   ` 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
  2 siblings, 0 replies; 12+ messages in thread
From: Paul Menzel @ 2010-11-05 22:55 UTC (permalink / raw)
  To: openembedded-devel

[-- Attachment #1: Type: text/plain, Size: 1406 bytes --]

Date: Thu, 4 Nov 2010 23:04:35 +0100

This fixes commit 93dca1 [1]. Thanks to Dallas for reporting this issue [2].

[1] http://cgit.openembedded.org/cgit.cgi/openembedded/commit/?id=93dca1c604d420ed4db44a71a375182933658886
[2] http://lists.linuxtogo.org/pipermail/openembedded-devel/2010-November/026475.html

Reported-by: Dallas Foley <dfoley@telus.net>
Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
---
I am deeply sorry for my mistake, missing QA and for the additional hassle.

The build still fails with the following error.

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

This problem is discussed in the above mentioned thread [2].
---
 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 ef764c1..af01d98 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 lua"
+DEPENDS += "libdvdcss libdvdread lua5.1"
 
 EXTRA_OECONF += "\
 	--enable-dvdread \
-- 
1.7.2.3

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* 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`)
  2010-11-03 17:55 ` Dallas Foley
  2010-11-03 20:08   ` Khem Raj
  2010-11-05 22:55   ` [PATCH] vlc_1.1.4.1: Really add `lua5.1` to `DEPENDS` (instead of just `lua`) Paul Menzel
@ 2010-11-05 23:21   ` Paul Menzel
  2010-11-06  7:51     ` Petr Štetiar
  2 siblings, 1 reply; 12+ messages in thread
From: Paul Menzel @ 2010-11-05 23:21 UTC (permalink / raw)
  To: openembedded-devel

[-- Attachment #1: Type: text/plain, Size: 4588 bytes --]

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@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

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: 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`)
  2010-11-05 23:21   ` 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
@ 2010-11-06  7:51     ` Petr Štetiar
  2010-11-06  9:31       ` vlc_1.1.4.1: Why is `lua5.1-native` not added to `DEPENDS` automatically? Paul Menzel
  0 siblings, 1 reply; 12+ messages in thread
From: Petr Štetiar @ 2010-11-06  7:51 UTC (permalink / raw)
  To: openembedded-devel

Paul Menzel <paulepanter@users.sourceforge.net> [2010-11-06 00:21:08]:

> Indeed in all my tests there is no `luac` staged.
> 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]?

The question is, why there's no luac for you :-) It's working for me (tm), see bellow. 

ynezz@ntbk:/media/data/devel/oe/ts72xx.git/tmp/work$ ls armv4t-angstrom-linux-*/lua5.1-5.1.4-r9/packages-split/lua5.1/usr/bin/*
armv4t-angstrom-linux-gnueabi/lua5.1-5.1.4-r9/packages-split/lua5.1/usr/bin/lua
armv4t-angstrom-linux-gnueabi/lua5.1-5.1.4-r9/packages-split/lua5.1/usr/bin/luac
armv4t-angstrom-linux-uclibceabi/lua5.1-5.1.4-r9/packages-split/lua5.1/usr/bin/lua
armv4t-angstrom-linux-uclibceabi/lua5.1-5.1.4-r9/packages-split/lua5.1/usr/bin/luac

-- ynezz



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: vlc_1.1.4.1: Why is `lua5.1-native` not added to `DEPENDS` automatically?
  2010-11-06  7:51     ` Petr Štetiar
@ 2010-11-06  9:31       ` Paul Menzel
  2010-11-06 10:36         ` Petr Štetiar
  0 siblings, 1 reply; 12+ messages in thread
From: Paul Menzel @ 2010-11-06  9:31 UTC (permalink / raw)
  To: openembedded-devel

[-- Attachment #1: Type: text/plain, Size: 1134 bytes --]

Am Samstag, den 06.11.2010, 08:51 +0100 schrieb Petr Štetiar:
> Paul Menzel <paulepanter@users.sourceforge.net> [2010-11-06 00:21:08]:
> 
> > Indeed in all my tests there is no `luac` staged.
> > 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]?
> 
> The question is, why there's no luac for you :-) It's working for me (tm), see bellow. 
> 
> ynezz@ntbk:/media/data/devel/oe/ts72xx.git/tmp/work$ ls armv4t-angstrom-linux-*/lua5.1-5.1.4-r9/packages-split/lua5.1/usr/bin/*
> armv4t-angstrom-linux-gnueabi/lua5.1-5.1.4-r9/packages-split/lua5.1/usr/bin/lua
> armv4t-angstrom-linux-gnueabi/lua5.1-5.1.4-r9/packages-split/lua5.1/usr/bin/luac
> armv4t-angstrom-linux-uclibceabi/lua5.1-5.1.4-r9/packages-split/lua5.1/usr/bin/lua
> armv4t-angstrom-linux-uclibceabi/lua5.1-5.1.4-r9/packages-split/lua5.1/usr/bin/luac

I think my question is, why `luac` is not staged if `vlc` `DEPENDS` on
`lua5.1`.

Or is is inevitable to add `lua5.1-native` to `DEPENDS`?


Thanks,

Paul

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: vlc_1.1.4.1: Why is `lua5.1-native` not added to `DEPENDS` automatically?
  2010-11-06  9:31       ` vlc_1.1.4.1: Why is `lua5.1-native` not added to `DEPENDS` automatically? Paul Menzel
@ 2010-11-06 10:36         ` Petr Štetiar
  2010-11-06 16:20           ` [PATCH] vlc_1.1.4.1: add `lua5.1-native` to `DEPENDS` and `lua5.1` to `RDEPENDS` Paul Menzel
  0 siblings, 1 reply; 12+ messages in thread
From: Petr Štetiar @ 2010-11-06 10:36 UTC (permalink / raw)
  To: openembedded-devel

[-- Attachment #1: Type: text/plain, Size: 247 bytes --]

Paul Menzel <paulepanter@users.sourceforge.net> [2010-11-06 10:31:25]:

> Or is is inevitable to add `lua5.1-native` to `DEPENDS`?

You have to. As an example I've attached for you my recipe for lua-lanes,
which uses luac also. 

-- ynezz

[-- Attachment #2: 0001-lua-lanes-add-new-recipe.patch --]
[-- Type: text/x-diff, Size: 11160 bytes --]

From 865463173cbb5d70282ea69c2c84084e435578e4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Petr=20=C5=A0tetiar?= <ynezz@true.cz>
Date: Sun, 10 Oct 2010 10:24:39 +0200
Subject: [PATCH] lua-lanes: add new recipe
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit


Signed-off-by: Petr Štetiar <ynezz@true.cz>
---
 .../0001-fix-LanesTimer-inifinite-thread-GC.patch  |   31 +++
 recipes/lua/lua-lanes-2.0.6/makefile.patch         |  207 ++++++++++++++++++++
 recipes/lua/lua-lanes-2.0.6/uclibc.patch           |   13 ++
 recipes/lua/lua-lanes_2.0.6.bb                     |   34 ++++
 4 files changed, 285 insertions(+), 0 deletions(-)
 create mode 100644 recipes/lua/lua-lanes-2.0.6/0001-fix-LanesTimer-inifinite-thread-GC.patch
 create mode 100644 recipes/lua/lua-lanes-2.0.6/makefile.patch
 create mode 100644 recipes/lua/lua-lanes-2.0.6/uclibc.patch
 create mode 100644 recipes/lua/lua-lanes_2.0.6.bb

diff --git a/recipes/lua/lua-lanes-2.0.6/0001-fix-LanesTimer-inifinite-thread-GC.patch b/recipes/lua/lua-lanes-2.0.6/0001-fix-LanesTimer-inifinite-thread-GC.patch
new file mode 100644
index 0000000..18fb36b
--- /dev/null
+++ b/recipes/lua/lua-lanes-2.0.6/0001-fix-LanesTimer-inifinite-thread-GC.patch
@@ -0,0 +1,31 @@
+From 69583ec6fc6cd18a0248d1233193bb2407dbda04 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Petr=20=C5=A0tetiar?= <ynezz@true.cz>
+Date: Sun, 31 Oct 2010 23:02:49 +0100
+Subject: [PATCH] fix LanesTimer inifinite thread GC
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+threading.c 394: pthread_cond_destroy(ref) failed, 16 EBUSY
+
+Signed-off-by: Petr Štetiar <ynezz@true.cz>
+---
+ src/lanes.c |    2 +-
+ 1 files changed, 1 insertions(+), 1 deletions(-)
+
+diff --git a/src/lanes.c b/src/lanes.c
+index c02c540..54cfa7a 100644
+--- a/src/lanes.c
++++ b/src/lanes.c
+@@ -1559,7 +1559,7 @@ LUAG_FUNC( thread_gc ) {
+ 
+     // We can read 's->status' without locks, but not wait for it
+     //
+-    if (s->status < DONE) {
++    if (s->status < DONE && strcmp(s->threadName, "LanesTimer")) {
+         //
+         selfdestruct_add(s);
+         assert( s->selfdestruct_next );
+-- 
+1.7.0.4
+
diff --git a/recipes/lua/lua-lanes-2.0.6/makefile.patch b/recipes/lua/lua-lanes-2.0.6/makefile.patch
new file mode 100644
index 0000000..6807010
--- /dev/null
+++ b/recipes/lua/lua-lanes-2.0.6/makefile.patch
@@ -0,0 +1,207 @@
+Index: lanes-2.0.6/src/Makefile
+===================================================================
+--- lanes-2.0.6.orig/src/Makefile	2009-01-22 07:32:18.000000000 +0100
++++ lanes-2.0.6/src/Makefile	2010-10-17 22:00:16.000000000 +0200
+@@ -11,123 +11,21 @@
+ 
+ OBJ=$(SRC:.c=.o)
+ 
+-# LuaRocks gives 'LIBFLAG' from the outside
+-#
+-LIBFLAG=-shared
+-
+-OPT_FLAGS=-O2
+-    # -O0 -g
+-
+ LUA=lua
+ LUAC=luac
+ 
+-_SO=.so
+-ifeq "$(findstring MINGW32,$(shell uname -s))" "MINGW32"
+-  _SO=.dll
+-endif
+-
+-ifeq "$(LUAROCKS)" ""
+-  ifeq "$(findstring MINGW32,$(shell uname -s))" "MINGW32"
+-    # MinGW MSYS on Windows
+-    #
+-    # - 'lua' and 'luac' expected to be on the path
+-    # - %LUA_DEV% must lead to include files and libraries (Lua for Windows >= 5.1.3.14)
+-    # - %MSCVR80% must be the full pathname of 'msvcr80.dll'
+-    #
+-    ifeq "$(LUA_DEV)" ""
+-      $(error LUA_DEV not defined - try i.e. 'make LUA_DEV=/c/Program\ Files/Lua/5.1')
+-    endif
+-    ifeq "$(MSVCR80)" ""
+-      MSVCR80:=$(LUA_DEV)/install/support/Microsoft.VC80.CRT.SP1/MSVCR80.DLL
+-      ifneq '$(shell test -f "$(MSVCR80)" && echo found)' 'found'
+-        $(error MSVCR80 not defined - set it to full path of msvcr80.dll')
+-      endif
+-      $(warning MSVCR80=$(MSVCR80))
+-    endif
+-    LUA_FLAGS:=-I "$(LUA_DEV)/include"
+-    LUA_LIBS:="$(LUA_DEV)/lua5.1.dll" -lgcc -lmsvcr80 "$(MSVCR80)"
+-    LIBFLAG=-shared -Wl,-Map,lanes.map
+-  else
+-    # Autodetect LUA_FLAGS and/or LUA_LIBS
+-    #
+-    ifneq "$(shell which pkg-config)" ""
+-      ifeq "$(shell pkg-config --exists lua5.1 && echo 1)" "1"
+-        LUA_FLAGS:=$(shell pkg-config --cflags lua5.1)
+-        LUA_LIBS:=$(shell pkg-config --libs lua5.1)
+-          #
+-          # Ubuntu: -I/usr/include/lua5.1 
+-          #         -llua5.1
+-      else
+-        ifeq "$(shell pkg-config --exists lua && echo 1)" "1"
+-          LUA_FLAGS:=$(shell pkg-config --cflags lua)
+-          LUA_LIBS:=$(shell pkg-config --libs lua)
+-            #
+-            # OS X fink with pkg-config:
+-            #      -I/sw/include 
+-            #      -L/sw/lib -llua -lm
+-        else
+-          $(warning *** 'pkg-config' existed but did not know of 'lua[5.1]' - Good luck!)
+-          LUA_FLAGS:=
+-          LUA_LIBS:=-llua
+-        endif
+-      endif
+-    else
+-      # No 'pkg-config'; try defaults
+-      #
+-      ifeq "$(shell uname -s)" "Darwin"
+-        $(warning *** Assuming 'fink' at default path)
+-        LUA_FLAGS:=-I/sw/include
+-        LUA_LIBS:=-L/sw/lib -llua
+-      else
+-        $(warning *** Assuming an arbitrary Lua installation; try installing 'pkg-config')
+-        LUA_FLAGS:=
+-        LUA_LIBS:=-llua
+-      endif
+-    endif
+-  endif
+-
+-  ifeq "$(shell uname -s)" "Darwin"
+-    # Some machines need 'MACOSX_DEPLOYMENT_TARGET=10.3' for using '-undefined dynamic_lookup'
+-    # (at least PowerPC running 10.4.11); does not harm the others
+-    #
+-    CC = MACOSX_DEPLOYMENT_TARGET=10.3 gcc
+-    LIBFLAG = -bundle -undefined dynamic_lookup
+-  endif
+-  
+-  CFLAGS=-Wall -Werror $(OPT_FLAGS) $(LUA_FLAGS)
+-  LIBS=$(LUA_LIBS)
+-endif
+-
+ #---
+-# PThread platform specifics
+-#
+-ifeq "$(shell uname -s)" "Linux"
+-  # -D_GNU_SOURCE needed for 'pthread_mutexattr_settype'
+-  CFLAGS += -D_GNU_SOURCE -fPIC
+-
+-  # Use of -DUSE_PTHREAD_TIMEDJOIN is possible, but not recommended (slower & keeps threads
+-  # unreleased somewhat longer)
+-  #CFLAGS += -DUSE_PTHREAD_TIMEDJOIN
+-
+-  LIBS += -lpthread
+-endif
+-
+-ifeq "$(shell uname -s)" "BSD"
+-  LIBS += -lpthread
+-endif
+-
+-#---
+-all: lua51-$(MODULE)$(_SO)
++all: lua51-$(MODULE).so
+ 
+ %.o: %.c *.h Makefile
+ 
+ # Note: Don't put $(LUA_LIBS) ahead of $^; MSYS will not like that (I think)
+ #
+-lua51-$(MODULE)$(_SO): $(OBJ)
+-	$(CC) $(LIBFLAG) $(LIBS) $^ $(LUA_LIBS) -o $@
++lua51-$(MODULE).so: $(OBJ)
++	$(CC) -shared -fPIC -lpthread -D_GNU_SOURCE `pkg-config --cflags --libs lua5.1` $(MYFLAGS) $^ -o $@
+ 
+ clean:
+-	-rm -rf lua51-$(MODULE)$(_SO) *.lch *.o *.tmp *.map
++	-rm -rf lua51-$(MODULE).so *.lch *.o *.tmp *.map
+ 
+ lanes.o: keeper.lch
+ 
+@@ -139,38 +37,5 @@
+ 	$(LUA) ../tools/bin2c.lua $@.tmp -o $@
+ 	-rm $@.tmp
+ 
+-#---
+-# NSLU2 "slug" Linux ARM
+-#
+-nslu2:
+-	$(MAKE) all CFLAGS="$(CFLAGS) -I/opt/include -L/opt/lib -D_GNU_SOURCE -lpthread"
+-
+-#---
+-# Cross compiling to Win32 (MinGW on OS X Intel)
+-#
+-# Point WIN32_LUA51 to an extraction of LuaBinaries dll8 and dev packages.
+-#
+-# Note: Only works on platforms with same endianess (i.e. not from PowerPC OS X,
+-#       since 'luac' uses the host endianess)
+-#
+-# EXPERIMENTAL; NOT TESTED OF LATE.
+-#
+-MINGW_GCC=mingw32-gcc
+-    # i686-pc-mingw32-gcc
+-
+-win32: $(WIN32_LUA51)/include/lua.h
+-	$(MAKE) build CC=$(MINGW_GCC) \
+-            LUA_FLAGS=-I$(WIN32_LUA51)/include \
+-            LUA_LIBS="-L$(WIN32_LUA51) -llua51" \
+-            _SO=.dll \
+-            SO_FLAGS=-shared \
+-            LUA=lua51 \
+-            LUAC=luac51
+-
+-$(WIN32_LUA51)/include/lua.h:
+-	@echo "Usage: make win32 WIN32_LUA51=<path of extracted LuaBinaries dll8 and dev packages>"
+-	@echo "                  [MINGW_GCC=...mingw32-gcc]"
+-	@false
+-
+-.PROXY:	all clean nslu2 win32
++.PROXY:	all clean
+ 
+Index: lanes-2.0.6/Makefile
+===================================================================
+--- lanes-2.0.6.orig/Makefile	2010-07-27 11:39:00.000000000 +0200
++++ lanes-2.0.6/Makefile	2010-10-17 22:14:20.000000000 +0200
+@@ -32,8 +32,10 @@
+ else
+   # Autodetect LUA & LUAC
+   #
+-  LUA=$(word 1,$(shell which lua5.1) $(shell which lua51) lua)
+-  LUAC=$(word 1,$(shell which luac5.1) $(shell which luac51) luac)
++  #LUA=$(word 1,$(shell which lua5.1) $(shell which lua51) lua)
++  #LUAC=$(word 1,$(shell which luac5.1) $(shell which luac51) luac)
++  LUA=lua
++  LUAC=luac
+ endif
+ 
+ _PREFIX=LUA_CPATH=./src/?$(_SO) LUA_PATH="src/?.lua;./tests/?.lua"
+@@ -42,7 +44,7 @@
+ all: $(_TARGET_SO)
+ 
+ $(_TARGET_SO): src/*.lua src/*.c src/*.h
+-	cd src && $(MAKE) LUA=$(LUA) LUAC=$(LUAC)
++	cd src && $(MAKE) LUA=$(LUA) LUAC=$(LUAC) MYFLAGS='$(MYFLAGS)'
+ 
+ clean:
+ 	cd src && $(MAKE) clean
+@@ -183,7 +185,7 @@
+ #
+ install: $(_TARGET_SO) src/lanes.lua
+ 	mkdir -p $(LUA_LIBDIR) $(LUA_SHAREDIR)
+-	install -m 644 $(_TARGET_SO) $(LUA_LIBDIR)
++	install $(_TARGET_SO) $(LUA_LIBDIR)
+ 	cp -p src/lanes.lua $(LUA_SHAREDIR)
+ 
+ 
diff --git a/recipes/lua/lua-lanes-2.0.6/uclibc.patch b/recipes/lua/lua-lanes-2.0.6/uclibc.patch
new file mode 100644
index 0000000..b3d1f83
--- /dev/null
+++ b/recipes/lua/lua-lanes-2.0.6/uclibc.patch
@@ -0,0 +1,13 @@
+Index: lanes-2.0.6/src/threading.h
+===================================================================
+--- lanes-2.0.6.orig/src/threading.h	2010-10-10 09:35:20.000000000 +0200
++++ lanes-2.0.6/src/threading.h	2010-10-10 09:35:32.000000000 +0200
+@@ -112,7 +112,7 @@
+   #ifdef PLATFORM_OSX
+     #define YIELD() pthread_yield_np()
+   #else
+-    #define YIELD() pthread_yield()
++    #define YIELD() sched_yield()
+   #endif
+ #endif
+ 
diff --git a/recipes/lua/lua-lanes_2.0.6.bb b/recipes/lua/lua-lanes_2.0.6.bb
new file mode 100644
index 0000000..49ceb9c
--- /dev/null
+++ b/recipes/lua/lua-lanes_2.0.6.bb
@@ -0,0 +1,34 @@
+DESCRIPTION = "Lua Lanes is a Lua extension library providing the possibility to run multiple Lua states in parallel."
+LICENSE = "MIT"
+HOMEPAGE = "http://kotisivu.dnainternet.net/askok/bin/lanes/index.html"
+
+DEPENDS += "lua5.1-native"
+RDEPENDS_${PN} += "lua5.1"
+
+PR = "r0"
+S = "${WORKDIR}/lanes-${PV}"
+
+SRC_URI = "http://luaforge.net/frs/download.php/4652/lanes-2.0.6.tgz \
+           file://makefile.patch \
+           file://0001-fix-LanesTimer-inifinite-thread-GC.patch \
+          "
+
+UCLIBC_PATCHES = "file://uclibc.patch"
+SRC_URI_append_linux-uclibc = "${UCLIBC_PATCHES}"
+SRC_URI_append_linux-uclibceabi = "${UCLIBC_PATCHES}"
+
+LUA_LIB_DIR =  "${libdir}/lua/5.1"
+LUA_SHARE_DIR = "${datadir}/lua/5.1"
+
+PACKAGES = "${PN} ${PN}-dbg"
+FILES_${PN} = "${LUA_LIB_DIR}/lua51-lanes.so ${LUA_SHARE_DIR}/lanes.lua"
+FILES_${PN}-dbg = "${LUA_LIB_DIR}/.debug/lua51-lanes.so"
+
+EXTRA_OEMAKE = "MYFLAGS='${CFLAGS} ${LDFLAGS}'"
+
+do_install() {
+	oe_runmake install DESTDIR='${D}/usr' 
+}
+
+SRC_URI[md5sum] = "1716bdf138fcc65b7069aaad864da677"
+SRC_URI[sha256sum] = "7b07fa1c21749981ec0addcf8e9e85acfca91efe8d3f64982a6ac0bc6a615e70"
-- 
1.7.0.4


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH] vlc_1.1.4.1: add `lua5.1-native` to `DEPENDS` and `lua5.1` to `RDEPENDS`
  2010-11-06 10:36         ` Petr Štetiar
@ 2010-11-06 16:20           ` Paul Menzel
  2010-11-11 22:26             ` [PATCH][RESENT] " Paul Menzel
  0 siblings, 1 reply; 12+ messages in thread
From: Paul Menzel @ 2010-11-06 16:20 UTC (permalink / raw)
  To: openembedded-devel

[-- Attachment #1: Type: text/plain, Size: 1878 bytes --]

Date: Sat, 6 Nov 2010 12:00:31 +0100

Task `configure` failed with the following error.

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

It turns out that `luac` is needed on the build system and therefore `lua5.1-native` needs to be added to `DEPENDS`. Thanks to Petr Štetiar for his advise on this issue [1].

With this patch applied I built this recipe successfully with the following build configuration.

        Build Configuration:
        BB_VERSION        = "1.10.0"
        METADATA_BRANCH   = "org.openembedded.dev"
        METADATA_REVISION = "28492a5"
        TARGET_ARCH       = "arm"
        TARGET_OS         = "linux-gnueabi"
        MACHINE           = "beagleboard"
        DISTRO            = "angstrom"
        DISTRO_VERSION    = "2010.7-test-20101106"
        TARGET_FPU        = "hard"

[1] http://lists.linuxtogo.org/pipermail/openembedded-devel/2010-November/026631.html

Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
---
Please apply after the first fixup patch [2].

[2] http://patchwork.openembedded.org/patch/3504/
---
 recipes/vlc/vlc_1.1.4.1.bb |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/recipes/vlc/vlc_1.1.4.1.bb b/recipes/vlc/vlc_1.1.4.1.bb
index af01d98..df8f17c 100644
--- a/recipes/vlc/vlc_1.1.4.1.bb
+++ b/recipes/vlc/vlc_1.1.4.1.bb
@@ -10,7 +10,8 @@ SRC_URI[sha256sum] = "61c9ea30a17ea40c6ccbfd507026e5c83ad9e0691f221d3667c8e49696
 
 # ffmpeg from git (library version => 52) is required
 # libtool-native must be >= 2.2.4
-DEPENDS += "libdvdcss libdvdread lua5.1"
+DEPENDS += "libdvdcss libdvdread lua5.1-native"
+RDEPENDS_${PN} += "lua5.1"
 
 EXTRA_OECONF += "\
 	--enable-dvdread \
-- 
1.7.2.3

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH][RESENT] vlc_1.1.4.1: add `lua5.1-native` to `DEPENDS` and `lua5.1` to `RDEPENDS`
  2010-11-06 16:20           ` [PATCH] vlc_1.1.4.1: add `lua5.1-native` to `DEPENDS` and `lua5.1` to `RDEPENDS` Paul Menzel
@ 2010-11-11 22:26             ` Paul Menzel
  2010-12-10 11:11               ` [PATCH][RESEND] " Paul Menzel
  0 siblings, 1 reply; 12+ messages in thread
From: Paul Menzel @ 2010-11-11 22:26 UTC (permalink / raw)
  To: openembedded-devel

[-- Attachment #1: Type: text/plain, Size: 1893 bytes --]

Date: Sat, 6 Nov 2010 12:00:31 +0100

Task `configure` failed with the following error.

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

It turns out that `luac` is needed on the build system and therefore `lua5.1-native` needs to be added to `DEPENDS`. Thanks to Petr Štetiar for his advise on this issue [1].

With this patch applied I built this recipe successfully with the following build configuration.

        Build Configuration:
        BB_VERSION        = "1.10.0"
        METADATA_BRANCH   = "org.openembedded.dev"
        METADATA_REVISION = "28492a5"
        TARGET_ARCH       = "arm"
        TARGET_OS         = "linux-gnueabi"
        MACHINE           = "beagleboard"
        DISTRO            = "angstrom"
        DISTRO_VERSION    = "2010.7-test-20101106"
        TARGET_FPU        = "hard"

[1] http://lists.linuxtogo.org/pipermail/openembedded-devel/2010-November/026631.html

Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
---
This patch has the number 3506 in the patch queue [1].

[1] http://patchwork.openembedded.org/patch/3506/
---
 recipes/vlc/vlc_1.1.4.1.bb |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/recipes/vlc/vlc_1.1.4.1.bb b/recipes/vlc/vlc_1.1.4.1.bb
index af01d98..df8f17c 100644
--- a/recipes/vlc/vlc_1.1.4.1.bb
+++ b/recipes/vlc/vlc_1.1.4.1.bb
@@ -10,7 +10,8 @@ SRC_URI[sha256sum] = "61c9ea30a17ea40c6ccbfd507026e5c83ad9e0691f221d3667c8e49696
 
 # ffmpeg from git (library version => 52) is required
 # libtool-native must be >= 2.2.4
-DEPENDS += "libdvdcss libdvdread lua5.1"
+DEPENDS += "libdvdcss libdvdread lua5.1-native"
+RDEPENDS_${PN} += "lua5.1"
 
 EXTRA_OECONF += "\
        --enable-dvdread \
-- 
1.7.2.3

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [PATCH][RESEND] vlc_1.1.4.1: add `lua5.1-native` to `DEPENDS` and `lua5.1` to `RDEPENDS`
  2010-11-11 22:26             ` [PATCH][RESENT] " Paul Menzel
@ 2010-12-10 11:11               ` Paul Menzel
  2010-12-11  8:33                 ` Khem Raj
  0 siblings, 1 reply; 12+ messages in thread
From: Paul Menzel @ 2010-12-10 11:11 UTC (permalink / raw)
  To: openembedded-devel

[-- Attachment #1: Type: text/plain, Size: 2130 bytes --]

Am Donnerstag, den 11.11.2010, 23:26 +0100 schrieb Paul Menzel:
> Date: Sat, 6 Nov 2010 12:00:31 +0100
> 
> Task `configure` failed with the following error.
> 
>         […]
>         checking for LUA... yes
>         checking for luac... no
>         configure: error: Could not find the LUA byte compiler.
>         ERROR: Function do_configure failed
> 
> It turns out that `luac` is needed on the build system and therefore `lua5.1-native` needs to be added to `DEPENDS`. Thanks to Petr Štetiar for his advise on this issue [1].
> 
> With this patch applied I built this recipe successfully with the following build configuration.
> 
>         Build Configuration:
>         BB_VERSION        = "1.10.0"
>         METADATA_BRANCH   = "org.openembedded.dev"
>         METADATA_REVISION = "28492a5"
>         TARGET_ARCH       = "arm"
>         TARGET_OS         = "linux-gnueabi"
>         MACHINE           = "beagleboard"
>         DISTRO            = "angstrom"
>         DISTRO_VERSION    = "2010.7-test-20101106"
>         TARGET_FPU        = "hard"
> 
> [1] http://lists.linuxtogo.org/pipermail/openembedded-devel/2010-November/026631.html
> 
> Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
> ---
> This patch has the number 3506 in the patch queue [1].
> 
> [1] http://patchwork.openembedded.org/patch/3506/

It would be great if some could commit and push this patch.


Thanks,

Paul

> ---
>  recipes/vlc/vlc_1.1.4.1.bb |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/recipes/vlc/vlc_1.1.4.1.bb b/recipes/vlc/vlc_1.1.4.1.bb
> index af01d98..df8f17c 100644
> --- a/recipes/vlc/vlc_1.1.4.1.bb
> +++ b/recipes/vlc/vlc_1.1.4.1.bb
> @@ -10,7 +10,8 @@ SRC_URI[sha256sum] = "61c9ea30a17ea40c6ccbfd507026e5c83ad9e0691f221d3667c8e49696
>  
>  # ffmpeg from git (library version => 52) is required
>  # libtool-native must be >= 2.2.4
> -DEPENDS += "libdvdcss libdvdread lua5.1"
> +DEPENDS += "libdvdcss libdvdread lua5.1-native"
> +RDEPENDS_${PN} += "lua5.1"
>  
>  EXTRA_OECONF += "\
>         --enable-dvdread \

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH][RESEND] vlc_1.1.4.1: add `lua5.1-native` to `DEPENDS` and `lua5.1` to `RDEPENDS`
  2010-12-10 11:11               ` [PATCH][RESEND] " Paul Menzel
@ 2010-12-11  8:33                 ` Khem Raj
  0 siblings, 0 replies; 12+ messages in thread
From: Khem Raj @ 2010-12-11  8:33 UTC (permalink / raw)
  To: openembedded-devel

On (10/12/10 12:11), Paul Menzel wrote:
> > ---
> > This patch has the number 3506 in the patch queue [1].
> > 
> > [1] http://patchwork.openembedded.org/patch/3506/
> 
> It would be great if some could commit and push this patch.

Applied thanks

> 
> 
> Thanks,
> 
> Paul
> 
> > ---
> >  recipes/vlc/vlc_1.1.4.1.bb |    3 ++-
> >  1 files changed, 2 insertions(+), 1 deletions(-)
> > 
> > diff --git a/recipes/vlc/vlc_1.1.4.1.bb b/recipes/vlc/vlc_1.1.4.1.bb
> > index af01d98..df8f17c 100644
> > --- a/recipes/vlc/vlc_1.1.4.1.bb
> > +++ b/recipes/vlc/vlc_1.1.4.1.bb
> > @@ -10,7 +10,8 @@ SRC_URI[sha256sum] = "61c9ea30a17ea40c6ccbfd507026e5c83ad9e0691f221d3667c8e49696
> >  
> >  # ffmpeg from git (library version => 52) is required
> >  # libtool-native must be >= 2.2.4
> > -DEPENDS += "libdvdcss libdvdread lua5.1"
> > +DEPENDS += "libdvdcss libdvdread lua5.1-native"
> > +RDEPENDS_${PN} += "lua5.1"
> >  
> >  EXTRA_OECONF += "\
> >         --enable-dvdread \



> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel




^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2010-12-11  8:35 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-11-03  9:28 [PATCH] vlc_1.1.4.1: add `lua5.1` to `DEPENDS` Paul Menzel
2010-11-03 17:55 ` Dallas Foley
2010-11-03 20:08   ` Khem Raj
2010-11-05 22:55   ` [PATCH] vlc_1.1.4.1: Really add `lua5.1` to `DEPENDS` (instead of just `lua`) Paul Menzel
2010-11-05 23:21   ` 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
2010-11-06  7:51     ` Petr Štetiar
2010-11-06  9:31       ` vlc_1.1.4.1: Why is `lua5.1-native` not added to `DEPENDS` automatically? Paul Menzel
2010-11-06 10:36         ` Petr Štetiar
2010-11-06 16:20           ` [PATCH] vlc_1.1.4.1: add `lua5.1-native` to `DEPENDS` and `lua5.1` to `RDEPENDS` Paul Menzel
2010-11-11 22:26             ` [PATCH][RESENT] " Paul Menzel
2010-12-10 11:11               ` [PATCH][RESEND] " Paul Menzel
2010-12-11  8:33                 ` Khem Raj

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.