All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 2/2] luajit: fix dependencies of Lua modules
@ 2012-06-01  9:58 François Perrad
  2012-06-01 16:20 ` Thomas Petazzoni
  0 siblings, 1 reply; 4+ messages in thread
From: François Perrad @ 2012-06-01  9:58 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
---
 package/cgilua/Config.in        |    1 -
 package/copas/Config.in         |    1 -
 package/lua/Config.in           |    3 +--
 package/luafilesystem/Config.in |    1 -
 package/luasocket/Config.in     |    1 -
 package/rings/Config.in         |    1 -
 package/xavante/Config.in       |    1 -
 7 files changed, 1 insertion(+), 8 deletions(-)

diff --git a/package/cgilua/Config.in b/package/cgilua/Config.in
index b43ba97..855ff48 100644
--- a/package/cgilua/Config.in
+++ b/package/cgilua/Config.in
@@ -1,6 +1,5 @@
 config BR2_PACKAGE_CGILUA
 	bool "cgilua"
-	select BR2_PACKAGE_LUA_SHARED_LIBRARY
 	select BR2_PACKAGE_LUAFILESYSTEM
 	help
 	  CGILua is a tool for creating dynamic HTML pages
diff --git a/package/copas/Config.in b/package/copas/Config.in
index 36abcc1..9969c8f 100644
--- a/package/copas/Config.in
+++ b/package/copas/Config.in
@@ -2,7 +2,6 @@ config BR2_PACKAGE_COPAS
 	bool "copas"
 	select BR2_PACKAGE_COXPCALL
 	select BR2_PACKAGE_LUASOCKET
-	select BR2_PACKAGE_LUA_SHARED_LIBRARY # for luasocket
 	help
 	  Copas is a dispatcher based on coroutines that
 	  can be used by TCP/IP servers.
diff --git a/package/lua/Config.in b/package/lua/Config.in
index 56eba35..b9cde26 100644
--- a/package/lua/Config.in
+++ b/package/lua/Config.in
@@ -1,5 +1,6 @@
 config BR2_PACKAGE_LUA
 	bool "lua"
+	select BR2_PACKAGE_LUA_SHARED_LIBRARY
 	help
 	  Lua is a powerful, fast, light-weight, embeddable scripting language.

@@ -8,14 +9,12 @@ config BR2_PACKAGE_LUA
 config BR2_PACKAGE_LUA_COMPILER
 	bool "lua compiler"
 	depends on BR2_PACKAGE_LUA
-	select BR2_PACKAGE_LUA_SHARED_LIBRARY
 	help
 	  Install luac binary

 config BR2_PACKAGE_LUA_INTERPRETER
 	bool "lua interpreter"
 	depends on BR2_PACKAGE_LUA
-	select BR2_PACKAGE_LUA_SHARED_LIBRARY
 	help
 	  Install lua binary

diff --git a/package/luafilesystem/Config.in b/package/luafilesystem/Config.in
index edcb414..7c130f8 100644
--- a/package/luafilesystem/Config.in
+++ b/package/luafilesystem/Config.in
@@ -1,6 +1,5 @@
 config BR2_PACKAGE_LUAFILESYSTEM
 	bool "luafilesystem"
-	select BR2_PACKAGE_LUA_SHARED_LIBRARY
 	help
 	  LuaFileSystem offers a portable way to access
 	  the underlying directory structure and file attributes.
diff --git a/package/luasocket/Config.in b/package/luasocket/Config.in
index 298db56..4526303 100644
--- a/package/luasocket/Config.in
+++ b/package/luasocket/Config.in
@@ -1,6 +1,5 @@
 config BR2_PACKAGE_LUASOCKET
 	bool "luasocket"
-	depends on BR2_PACKAGE_LUA_SHARED_LIBRARY
 	help
 	  LuaSocket is the most comprehensive networking support library
 	  for the Lua language.
diff --git a/package/rings/Config.in b/package/rings/Config.in
index 2fe14aa..d1f1efe 100644
--- a/package/rings/Config.in
+++ b/package/rings/Config.in
@@ -1,6 +1,5 @@
 config BR2_PACKAGE_RINGS
 	bool "rings"
-	depends on BR2_PACKAGE_LUA_SHARED_LIBRARY
 	help
 	  Provides a way to create new Lua states from within Lua.

diff --git a/package/xavante/Config.in b/package/xavante/Config.in
index a7df530..db34d66 100644
--- a/package/xavante/Config.in
+++ b/package/xavante/Config.in
@@ -1,6 +1,5 @@
 config BR2_PACKAGE_XAVANTE
 	bool "xavante"
-	select BR2_PACKAGE_LUA_SHARED_LIBRARY
 	select BR2_PACKAGE_CGILUA
 	select BR2_PACKAGE_COPAS
 	select BR2_PACKAGE_COXPCALL
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 2/2] luajit: fix dependencies of Lua modules
  2012-06-01  9:58 [Buildroot] [PATCH 2/2] luajit: fix dependencies of Lua modules François Perrad
@ 2012-06-01 16:20 ` Thomas Petazzoni
  2012-06-01 18:50   ` François Perrad
  0 siblings, 1 reply; 4+ messages in thread
From: Thomas Petazzoni @ 2012-06-01 16:20 UTC (permalink / raw)
  To: buildroot

Le Fri, 1 Jun 2012 11:58:27 +0200,
Fran?ois Perrad <francois.perrad@gadz.org> a ?crit :

> Signed-off-by: Francois Perrad <francois.perrad@gadz.org>

I am not a Lua guy, so maybe I'm missing something obvious, but you
could expand the justification for this patch, and include this
justification in the commit log?

Thanks!

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

^ permalink raw reply	[flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 2/2] luajit: fix dependencies of Lua modules
  2012-06-01 16:20 ` Thomas Petazzoni
@ 2012-06-01 18:50   ` François Perrad
  2012-07-13  8:36     ` François Perrad
  0 siblings, 1 reply; 4+ messages in thread
From: François Perrad @ 2012-06-01 18:50 UTC (permalink / raw)
  To: buildroot

2012/6/1 Thomas Petazzoni <thomas.petazzoni@free-electrons.com>:
> Le Fri, 1 Jun 2012 11:58:27 +0200,
> Fran?ois Perrad <francois.perrad@gadz.org> a ?crit :
>
>> Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
>
> I am not a Lua guy, so maybe I'm missing something obvious, but you
> could expand the justification for this patch, and include this
> justification in the commit log?
>

luajit is another Lua 5.1 interpreter, API- and ABI-compatible to the
standard one (lua package).
A Lua module (luasocket for example) depends on lua OR luajit (only
one is deployed on the target).
The dependency against BR2_PACKAGE_LUA_SHARED_LIBRARY must be removed,
because it's an internal of lua package.


Fran?ois

> Thanks!
>
> Thomas
> --
> Thomas Petazzoni, Free Electrons
> Kernel, drivers, real-time and embedded Linux
> development, consulting, training and support.
> http://free-electrons.com
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 2/2] luajit: fix dependencies of Lua modules
  2012-06-01 18:50   ` François Perrad
@ 2012-07-13  8:36     ` François Perrad
  0 siblings, 0 replies; 4+ messages in thread
From: François Perrad @ 2012-07-13  8:36 UTC (permalink / raw)
  To: buildroot

2012/6/1 Fran?ois Perrad <francois.perrad@gadz.org>:
> 2012/6/1 Thomas Petazzoni <thomas.petazzoni@free-electrons.com>:
>> Le Fri, 1 Jun 2012 11:58:27 +0200,
>> Fran?ois Perrad <francois.perrad@gadz.org> a ?crit :
>>
>>> Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
>>
>> I am not a Lua guy, so maybe I'm missing something obvious, but you
>> could expand the justification for this patch, and include this
>> justification in the commit log?
>>
>
> luajit is another Lua 5.1 interpreter, API- and ABI-compatible to the
> standard one (lua package).
> A Lua module (luasocket for example) depends on lua OR luajit (only
> one is deployed on the target).
> The dependency against BR2_PACKAGE_LUA_SHARED_LIBRARY must be removed,
> because it's an internal of lua package.
>
>

As side effect, this patch solves the building issue of lua in
autobuild.builroot.net.

Fran?ois

> Fran?ois
>
>> Thanks!
>>
>> Thomas
>> --
>> Thomas Petazzoni, Free Electrons
>> Kernel, drivers, real-time and embedded Linux
>> development, consulting, training and support.
>> http://free-electrons.com
>> _______________________________________________
>> buildroot mailing list
>> buildroot at busybox.net
>> http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-07-13  8:36 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-06-01  9:58 [Buildroot] [PATCH 2/2] luajit: fix dependencies of Lua modules François Perrad
2012-06-01 16:20 ` Thomas Petazzoni
2012-06-01 18:50   ` François Perrad
2012-07-13  8:36     ` François Perrad

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.