Message ID | 1389705298-27263-4-git-send-email-francois.perrad@gadz.org |
---|---|
State | Superseded |
Headers | show |
On 14/01/14 14:14, Francois Perrad wrote: > Signed-off-by: Francois Perrad<francois.perrad@gadz.org> > --- > package/lbase64/Config.in | 4 ++++ > package/lua-ev/lua-ev.mk | 2 +- > package/lua-msgpack-native/Config.in | 4 ++++ > package/lua-msgpack-native/lua-msgpack-native.mk | 3 ++- > package/luaexpat/Config.in | 4 ++++ > package/luaexpatutils/Config.in | 4 ++++ > package/luaexpatutils/luaexpatutils.mk | 2 +- > package/luaposix/Config.in | 2 +- > package/luaposix/luaposix.mk | 2 +- > package/luarocks/luarocks.mk | 4 ++-- > package/luasec/Config.in | 4 ++++ I wonder if this patch shouldn't be squashed with the first one - I guess without this patch, all these packages are broken when 5.2 is selected. > 11 files changed, 28 insertions(+), 7 deletions(-) > > diff --git a/package/lbase64/Config.in b/package/lbase64/Config.in > index ca9dd5d..9cc2e2b 100644 > --- a/package/lbase64/Config.in > +++ b/package/lbase64/Config.in > @@ -1,7 +1,11 @@ > config BR2_PACKAGE_LBASE64 > bool "lbase64" > depends on BR2_PACKAGE_HAS_LUA_INTERPRETER > + depends on !BR2_PACKAGE_LUA_5_2 I think this should be "depends on BR2_PACKAGE_LUA_5_1", because this package most likely still won't work when a Lua 5.3 is added in the future. But of course, to make that possible you need a BR2_PACKAGE_LUAINTERPRETER_ABI_5_1 symbol. > help > A base64 library for Lua > > http://luaforge.net/projects/lbase64/ > + > +comment "lbase64 only with Lua 5.1" > + depends on BR2_PACKAGE_LUA_5_2 To make it consistent with the toolchain comments, I'd write: comment "lbase64 needs a Lua 5.1 interpreter" Regards, Arnout [snip]
2014/1/14 Arnout Vandecappelle <arnout@mind.be>: > On 14/01/14 14:14, Francois Perrad wrote: >> >> Signed-off-by: Francois Perrad<francois.perrad@gadz.org> >> --- >> package/lbase64/Config.in | 4 ++++ >> package/lua-ev/lua-ev.mk | 2 +- >> package/lua-msgpack-native/Config.in | 4 ++++ >> package/lua-msgpack-native/lua-msgpack-native.mk | 3 ++- >> package/luaexpat/Config.in | 4 ++++ >> package/luaexpatutils/Config.in | 4 ++++ >> package/luaexpatutils/luaexpatutils.mk | 2 +- >> package/luaposix/Config.in | 2 +- >> package/luaposix/luaposix.mk | 2 +- >> package/luarocks/luarocks.mk | 4 ++-- >> package/luasec/Config.in | 4 ++++ > > > I wonder if this patch shouldn't be squashed with the first one - I guess > without this patch, all these packages are broken when 5.2 is selected. > > >> 11 files changed, 28 insertions(+), 7 deletions(-) >> >> diff --git a/package/lbase64/Config.in b/package/lbase64/Config.in >> index ca9dd5d..9cc2e2b 100644 >> --- a/package/lbase64/Config.in >> +++ b/package/lbase64/Config.in >> @@ -1,7 +1,11 @@ >> config BR2_PACKAGE_LBASE64 >> bool "lbase64" >> depends on BR2_PACKAGE_HAS_LUA_INTERPRETER >> + depends on !BR2_PACKAGE_LUA_5_2 > > > I think this should be "depends on BR2_PACKAGE_LUA_5_1", because this > package most likely still won't work when a Lua 5.3 is added in the future. > But of course, to make that possible you need a > BR2_PACKAGE_LUAINTERPRETER_ABI_5_1 symbol. > > >> help >> A base64 library for Lua >> >> http://luaforge.net/projects/lbase64/ >> + >> +comment "lbase64 only with Lua 5.1" >> + depends on BR2_PACKAGE_LUA_5_2 > > > To make it consistent with the toolchain comments, I'd write: > > comment "lbase64 needs a Lua 5.1 interpreter" Ok. François > > > Regards, > Arnout > > [snip] > -- > Arnout Vandecappelle arnout at mind be > Senior Embedded Software Architect +32-16-286500 > Essensium/Mind http://www.mind.be > G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven > LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle > GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot
diff --git a/package/lbase64/Config.in b/package/lbase64/Config.in index ca9dd5d..9cc2e2b 100644 --- a/package/lbase64/Config.in +++ b/package/lbase64/Config.in @@ -1,7 +1,11 @@ config BR2_PACKAGE_LBASE64 bool "lbase64" depends on BR2_PACKAGE_HAS_LUA_INTERPRETER + depends on !BR2_PACKAGE_LUA_5_2 help A base64 library for Lua http://luaforge.net/projects/lbase64/ + +comment "lbase64 only with Lua 5.1" + depends on BR2_PACKAGE_LUA_5_2 diff --git a/package/lua-ev/lua-ev.mk b/package/lua-ev/lua-ev.mk index 1236d77..a4f7964 100644 --- a/package/lua-ev/lua-ev.mk +++ b/package/lua-ev/lua-ev.mk @@ -9,6 +9,6 @@ LUA_EV_SITE = $(call github,brimworks,lua-ev,$(LUA_EV_VERSION)) LUA_EV_DEPENDENCIES = luainterpreter libev LUA_EV_LICENSE = MIT LUA_EV_LICENSE_FILES = README -LUA_EV_CONF_OPT = -DINSTALL_CMOD="/usr/lib/lua/5.1" +LUA_EV_CONF_OPT = -DINSTALL_CMOD="/usr/lib/lua/$(LUA_ABIVER)" $(eval $(cmake-package)) diff --git a/package/lua-msgpack-native/Config.in b/package/lua-msgpack-native/Config.in index 1014298..735c7a6 100644 --- a/package/lua-msgpack-native/Config.in +++ b/package/lua-msgpack-native/Config.in @@ -1,8 +1,12 @@ config BR2_PACKAGE_LUA_MSGPACK_NATIVE bool "lua-msgpack-native" depends on BR2_PACKAGE_HAS_LUA_INTERPRETER + depends on !BR2_PACKAGE_LUA_5_2 help This is a native, C language implementation of msgpack (http://msgpack.org) library/protocol for Lua language https://github.com/kengonakajima/lua-msgpack-native + +comment "lua-msgpack-native only with Lua 5.1" + depends on BR2_PACKAGE_LUA_5_2 diff --git a/package/lua-msgpack-native/lua-msgpack-native.mk b/package/lua-msgpack-native/lua-msgpack-native.mk index d4547ae..76ce037 100644 --- a/package/lua-msgpack-native/lua-msgpack-native.mk +++ b/package/lua-msgpack-native/lua-msgpack-native.mk @@ -15,7 +15,8 @@ define LUA_MSGPACK_NATIVE_BUILD_CMDS endef define LUA_MSGPACK_NATIVE_INSTALL_TARGET_CMDS - $(INSTALL) -m 755 -D $(@D)/msgpack.so $(TARGET_DIR)/usr/lib/lua/5.1/msgpack.so + $(INSTALL) -m 755 -D $(@D)/msgpack.so \ + $(TARGET_DIR)/usr/lib/lua/$(LUA_ABIVER)/msgpack.so endef $(eval $(generic-package)) diff --git a/package/luaexpat/Config.in b/package/luaexpat/Config.in index f8db318..38198e2 100644 --- a/package/luaexpat/Config.in +++ b/package/luaexpat/Config.in @@ -2,7 +2,11 @@ config BR2_PACKAGE_LUAEXPAT bool "luaexpat" select BR2_PACKAGE_EXPAT depends on BR2_PACKAGE_HAS_LUA_INTERPRETER + depends on !BR2_PACKAGE_LUA_5_2 help LuaExpat is a SAX XML parser based on the Expat library. http://matthewwild.co.uk/projects/luaexpat/ + +comment "LuaExpat only with Lua 5.1" + depends on BR2_PACKAGE_LUA_5_2 diff --git a/package/luaexpatutils/Config.in b/package/luaexpatutils/Config.in index cb2887a..213f371 100644 --- a/package/luaexpatutils/Config.in +++ b/package/luaexpatutils/Config.in @@ -1,8 +1,12 @@ config BR2_PACKAGE_LUAEXPATUTILS bool "luaexpatutils" select BR2_PACKAGE_LUAEXPAT + depends on !BR2_PACKAGE_LUA_5_2 help lxp.doc is a module that provides useful features for working with XML formats in LOM format as used by the LuaExpat project from Kepler. https://github.com/stevedonovan/LuaExpatUtils + +comment "LuaExpatUtils only with Lua 5.1" + depends on BR2_PACKAGE_LUA_5_2 diff --git a/package/luaexpatutils/luaexpatutils.mk b/package/luaexpatutils/luaexpatutils.mk index 93cf621..05a0b81 100644 --- a/package/luaexpatutils/luaexpatutils.mk +++ b/package/luaexpatutils/luaexpatutils.mk @@ -11,7 +11,7 @@ LUAEXPATUTILS_DEPENDENCIES = luaexpat define LUAEXPATUTILS_INSTALL_TARGET_CMDS $(INSTALL) -m 0644 -D $(@D)/lua/doc.lua \ - $(TARGET_DIR)/usr/share/lua/5.1/lxp/doc.lua + $(TARGET_DIR)/usr/share/lua/$(LUA_ABIVER)/lxp/doc.lua endef $(eval $(generic-package)) diff --git a/package/luaposix/Config.in b/package/luaposix/Config.in index cad6fe9..a95c825 100644 --- a/package/luaposix/Config.in +++ b/package/luaposix/Config.in @@ -1,7 +1,7 @@ config BR2_PACKAGE_LUAPOSIX bool "luaposix" depends on BR2_PACKAGE_HAS_LUA_INTERPRETER - select BR2_PACKAGE_LUABITOP if BR2_PACKAGE_LUA + select BR2_PACKAGE_LUABITOP if BR2_PACKAGE_LUA_5_1 # LuaBitOp is already included in LuaJIT help luaposix is a POSIX binding, including curses, for Lua 5.1 diff --git a/package/luaposix/luaposix.mk b/package/luaposix/luaposix.mk index b018995..6d13db5 100644 --- a/package/luaposix/luaposix.mk +++ b/package/luaposix/luaposix.mk @@ -10,7 +10,7 @@ LUAPOSIX_SOURCE = release-v$(LUAPOSIX_VERSION).tar.gz LUAPOSIX_LICENSE = MIT LUAPOSIX_LICENSE_FILES = COPYING LUAPOSIX_DEPENDENCIES = luainterpreter host-lua -LUAPOSIX_CONF_OPT = --libdir="/usr/lib/lua/5.1" --datarootdir="/usr/share/lua/5.1" +LUAPOSIX_CONF_OPT = --libdir="/usr/lib/lua/$(LUA_ABIVER)" --datarootdir="/usr/share/lua/$(LUA_ABIVER)" ifeq ($(BR2_PACKAGE_NCURSES),y) LUAPOSIX_DEPENDENCIES += ncurses diff --git a/package/luarocks/luarocks.mk b/package/luarocks/luarocks.mk index 85b6866..034a792 100644 --- a/package/luarocks/luarocks.mk +++ b/package/luarocks/luarocks.mk @@ -12,7 +12,7 @@ LUAROCKS_LICENSE_FILES = COPYING HOST_LUAROCKS_DEPENDENCIES = host-lua luainterpreter LUAROCKS_CONFIG_DIR = $(HOST_DIR)/usr/etc/luarocks -LUAROCKS_CONFIG_FILE = $(LUAROCKS_CONFIG_DIR)/config-5.1.lua +LUAROCKS_CONFIG_FILE = $(LUAROCKS_CONFIG_DIR)/config-$(LUA_ABIVER).lua define HOST_LUAROCKS_CONFIGURE_CMDS cd $(@D) && ./configure \ @@ -41,5 +41,5 @@ endef $(eval $(host-generic-package)) -LUAROCKS_RUN = LUA_PATH="$(HOST_DIR)/usr/share/lua/5.1/?.lua" \ +LUAROCKS_RUN = LUA_PATH="$(HOST_DIR)/usr/share/lua/$(LUA_ABIVER)/?.lua" \ $(HOST_DIR)/usr/bin/lua $(HOST_DIR)/usr/bin/luarocks diff --git a/package/luasec/Config.in b/package/luasec/Config.in index ed55df7..46a7c9c 100644 --- a/package/luasec/Config.in +++ b/package/luasec/Config.in @@ -3,8 +3,12 @@ config BR2_PACKAGE_LUASEC select BR2_PACKAGE_OPENSSL select BR2_PACKAGE_LUASOCKET depends on BR2_PACKAGE_HAS_LUA_INTERPRETER + depends on !BR2_PACKAGE_LUA_5_2 help LuaSec is a binding for OpenSSL library to provide TLS/SSL communication. http://www.inf.puc-rio.br/~brunoos/luasec/ + +comment "LuaSec only with Lua 5.1" + depends on BR2_PACKAGE_LUA_5_2
Signed-off-by: Francois Perrad <francois.perrad@gadz.org> --- package/lbase64/Config.in | 4 ++++ package/lua-ev/lua-ev.mk | 2 +- package/lua-msgpack-native/Config.in | 4 ++++ package/lua-msgpack-native/lua-msgpack-native.mk | 3 ++- package/luaexpat/Config.in | 4 ++++ package/luaexpatutils/Config.in | 4 ++++ package/luaexpatutils/luaexpatutils.mk | 2 +- package/luaposix/Config.in | 2 +- package/luaposix/luaposix.mk | 2 +- package/luarocks/luarocks.mk | 4 ++-- package/luasec/Config.in | 4 ++++ 11 files changed, 28 insertions(+), 7 deletions(-)