Skip to contentSkip to conversation

ljsyscall: remove dependency on some architecture

PATCH New Francois Perrad 2 participants 3 messages Mar 21, 2015 – Mar 23, 2015

lore
Share & tools
lore

Francois Perrad

author
#0 lore raw

PATCH v2 1/2 ljsyscall: remove dependency on some architecture

ljsyscall is available for all arch supported by LuaJIT. refactor LJSYSCALL_ARCH : dispatch all arch.

Signed-off-by Francois Perrad <[email protected]>
---
 package/ljsyscall/Config.in    | 4 +---
 package/ljsyscall/ljsyscall.mk | 7 ++++++-
 2 files changed, 7 insertions(+), 4 deletions(-)
package/ljsyscall/Config.in +1 -3
index dcbd521..149e71d 100644
@@ -2,13 +2,11 @@ config BR2_PACKAGE_LJSYSCALL
 	bool "ljsyscall"
 	# ljsyscall is specifically for LuaJIT, not Lua.
 	depends on BR2_PACKAGE_LUAJIT
-	# ljsyscall is only available for some target architectures
-	depends on BR2_i386 || BR2_x86_64 || BR2_powerpc || BR2_arm || BR2_armeb
 	help
 	  An FFI implementation of the Linux and NetBSD kernel ABIs for LuaJIT.
 
 	  http://github.com/justincormack/ljsyscall
 
 comment "ljsyscall needs LuaJIT"
-	depends on !BR2_PACKAGE_LUAJIT && (BR2_i386 || BR2_x86_64 || BR2_powerpc || BR2_arm || BR2_armeb)
+	depends on !BR2_PACKAGE_LUAJIT
package/ljsyscall/ljsyscall.mk +6 -1
index a17da3f..1c6c2c1 100644
@@ -9,13 +9,18 @@ LJSYSCALL_SITE = $(call github,justincormack,ljsyscall,$(LJSYSCALL_VERSION))
 LJSYSCALL_LICENSE = MIT
 LJSYSCALL_LICENSE_FILES = COPYRIGHT
 
+# dispatch all architectures of LuaJIT
 ifeq ($(BR2_i386),y)
 LJSYSCALL_ARCH = x86
 else ifeq ($(BR2_x86_64),y)
 LJSYSCALL_ARCH = x64
 else ifeq ($(BR2_powerpc),y)
 LJSYSCALL_ARCH = ppc
-else ifeq ($(LJSYSCALL_ARCH),)
+else ifeq ($(BR2_arm)$(BR2_armeb),y)
+LJSYSCALL_ARCH = arm
+else ifeq ($(BR2_mips)$(BR2_mipsel),y)
+LJSYSCALL_ARCH = mips
+else
 LJSYSCALL_ARCH = $(BR2_ARCH)
 endif
Signature · 1 line
2.1.0
author
#1 lore raw

PATCH v2 2/2 ljlinenoise: remove dependency on some architecture

ljsyscall is available for all arch supported by LuaJIT.

Signed-off-by Francois Perrad <[email protected]>
---
 package/ljlinenoise/Config.in | 3 ---
 1 file changed, 3 deletions(-)
package/ljlinenoise/Config.in +0 -3
index 17be1be..be83bdb 100644
@@ -4,8 +4,6 @@ config BR2_PACKAGE_LJLINENOISE
 	select BR2_PACKAGE_LJSYSCALL
 	# ljsyscall is specifically for LuaJIT, not Lua.
 	depends on BR2_PACKAGE_LUAJIT
-	# ljsyscall is only available for some target architectures
-	depends on BR2_i386 || BR2_x86_64 || BR2_powerpc || BR2_arm || BR2_armeb
 	help
 	  ljlinenoise is a pure LuaJIT port of linenoise,
 	  a small alternative to readline and libedit.

@@ -14,4 +12,3 @@ config BR2_PACKAGE_LJLINENOISE
 
 comment "ljlinenoise needs LuaJIT"
 	depends on !BR2_PACKAGE_LUAJIT
-	depends on BR2_i386 || BR2_x86_64 || BR2_powerpc || BR2_arm || BR2_armeb
Signature · 1 line
2.1.0
#2 lore raw

PATCH v2 1/2 ljsyscall: remove dependency on some architecture

In reply to Francois Perrad

Dear Francois Perrad,

Quoted 4 lines · Francois Perrad

ljsyscall is available for all arch supported by LuaJIT. refactor LJSYSCALL_ARCH : dispatch all arch.

Signed-off-by Francois Perrad <[email protected]>

Thanks, both patches applied.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com