[Buildroot] [PATCH 1/3] luarocks: bump to version 2.2.1

[Buildroot] [PATCH 1/3] luarocks: bump to version 2.2.1

[Francois Perrad]

Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
---
 package/luarocks/luarocks.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/luarocks/luarocks.mk b/package/luarocks/luarocks.mk
index 40fdc91..12a1f08 100644
--- a/package/luarocks/luarocks.mk
+++ b/package/luarocks/luarocks.mk
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-LUAROCKS_VERSION = 2.2.0
+LUAROCKS_VERSION = 2.2.1
 LUAROCKS_SITE = http://luarocks.org/releases
 LUAROCKS_LICENSE = MIT
 LUAROCKS_LICENSE_FILES = COPYING
-- 
2.1.0

[Buildroot] [PATCH 2/3] ljlinenoise: refactor with BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS

[Francois Perrad]

Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
---
 package/ljlinenoise/Config.in | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/package/ljlinenoise/Config.in b/package/ljlinenoise/Config.in
index 17be1be..ce1a156 100644
--- a/package/ljlinenoise/Config.in
+++ b/package/ljlinenoise/Config.in
@@ -5,7 +5,7 @@ config BR2_PACKAGE_LJLINENOISE
 	# 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
+	depends on BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS
 	help
 	  ljlinenoise is a pure LuaJIT port of linenoise,
 	  a small alternative to readline and libedit.
@@ -13,5 +13,4 @@ config BR2_PACKAGE_LJLINENOISE
 	  http://fperrad.github.io/ljlinenoise/
 
 comment "ljlinenoise needs LuaJIT"
-	depends on !BR2_PACKAGE_LUAJIT
-	depends on BR2_i386 || BR2_x86_64 || BR2_powerpc || BR2_arm || BR2_armeb
+	depends on !BR2_PACKAGE_LUAJIT && !BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS
-- 
2.1.0

[Buildroot] [PATCH 3/3] ljsyscall: refactor with BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS

[Francois Perrad]

Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
---
 package/ljsyscall/Config.in | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/ljsyscall/Config.in b/package/ljsyscall/Config.in
index dcbd521..b9ff04d 100644
--- a/package/ljsyscall/Config.in
+++ b/package/ljsyscall/Config.in
@@ -3,12 +3,12 @@ config 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
+	depends on BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS
 	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 && !BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS
 
-- 
2.1.0

[Buildroot] [PATCH 1/3] luarocks: bump to version 2.2.1

[Thomas Petazzoni]

Dear Francois Perrad,

On Tue, 17 Mar 2015 21:04:47 +0100, Francois Perrad wrote:
> Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
> ---
>  package/luarocks/luarocks.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied, thanks!

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

[Buildroot] [PATCH 2/3] ljlinenoise: refactor with BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS

[Thomas Petazzoni]

Dear Francois Perrad,

On Tue, 17 Mar 2015 21:04:48 +0100, Francois Perrad wrote:
> Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
> ---
>  package/ljlinenoise/Config.in | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/package/ljlinenoise/Config.in b/package/ljlinenoise/Config.in
> index 17be1be..ce1a156 100644
> --- a/package/ljlinenoise/Config.in
> +++ b/package/ljlinenoise/Config.in
> @@ -5,7 +5,7 @@ config BR2_PACKAGE_LJLINENOISE
>  	# 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
> +	depends on BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS

Are you sure?

BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS is clearly not the same as "BR2_i386
|| BR2_x86_64 || BR2_powerpc || BR2_arm || BR2_armeb". For example,
mips/mipsel are supported in LuaJIT, but not in the current condition
of ljlinenoise.

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

[Buildroot] [PATCH 3/3] ljsyscall: refactor with BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS

[Angelo Compagnucci]

Dear Francois Perrad,

2015-03-17 21:04 GMT+01:00 Francois Perrad <fperrad@gmail.com>:

Reviewed-by: Angelo Compagnucci <angelo.compagnucci@gmail.com>
> Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
> ---

Patch is fine for me. Nothing to add.

>  package/ljsyscall/Config.in | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/package/ljsyscall/Config.in b/package/ljsyscall/Config.in
> index dcbd521..b9ff04d 100644
> --- a/package/ljsyscall/Config.in
> +++ b/package/ljsyscall/Config.in
> @@ -3,12 +3,12 @@ config 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
> +       depends on BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS
>         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 && !BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS
>
> --
> 2.1.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot



-- 
Profile: http://it.linkedin.com/in/compagnucciangelo

[Buildroot] [PATCH 3/3] ljsyscall: refactor with BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS

[Angelo Compagnucci]

Dear Francois Perrad,


2015-03-17 22:45 GMT+01:00 Angelo Compagnucci <angelo.compagnucci@gmail.com>:
> Dear Francois Perrad,
>
> 2015-03-17 21:04 GMT+01:00 Francois Perrad <fperrad@gmail.com>:
>
> Reviewed-by: Angelo Compagnucci <angelo.compagnucci@gmail.com>
>> Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
>> ---
>
> Patch is fine for me. Nothing to add.
>
>>  package/ljsyscall/Config.in | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/package/ljsyscall/Config.in b/package/ljsyscall/Config.in
>> index dcbd521..b9ff04d 100644
>> --- a/package/ljsyscall/Config.in
>> +++ b/package/ljsyscall/Config.in
>> @@ -3,12 +3,12 @@ config 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
>> +       depends on BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS

Sorry for not having spotted before but

BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS != BR2_i386 || BR2_x86_64 ||
BR2_powerpc || BR2_arm || BR2_armeb

Actually, BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS is BR2_i386 has this condition more

((BR2_mips || BR2_mipsel) && !BR2_MIPS_SOFT_FLOAT)

Are you sure LJSYSCALL package works on mips?

>>         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 && !BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS
>>
>> --
>> 2.1.0
>>
>> _______________________________________________
>> buildroot mailing list
>> buildroot at busybox.net
>> http://lists.busybox.net/mailman/listinfo/buildroot
>
>
>
> --
> Profile: http://it.linkedin.com/in/compagnucciangelo



-- 
Profile: http://it.linkedin.com/in/compagnucciangelo

[Buildroot] [PATCH 3/3] ljsyscall: refactor with BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS

[François Perrad]

2015-03-17 23:37 GMT+01:00 Angelo Compagnucci <angelo.compagnucci@gmail.com>
:

> Dear Francois Perrad,
>
>
> 2015-03-17 22:45 GMT+01:00 Angelo Compagnucci <
> angelo.compagnucci at gmail.com>:
> > Dear Francois Perrad,
> >
> > 2015-03-17 21:04 GMT+01:00 Francois Perrad <fperrad@gmail.com>:
> >
> > Reviewed-by: Angelo Compagnucci <angelo.compagnucci@gmail.com>
> >> Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
> >> ---
> >
> > Patch is fine for me. Nothing to add.
> >
> >>  package/ljsyscall/Config.in | 4 ++--
> >>  1 file changed, 2 insertions(+), 2 deletions(-)
> >>
> >> diff --git a/package/ljsyscall/Config.in b/package/ljsyscall/Config.in
> >> index dcbd521..b9ff04d 100644
> >> --- a/package/ljsyscall/Config.in
> >> +++ b/package/ljsyscall/Config.in
> >> @@ -3,12 +3,12 @@ config 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
> >> +       depends on BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS
>
> Sorry for not having spotted before but
>
> BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS != BR2_i386 || BR2_x86_64 ||
> BR2_powerpc || BR2_arm || BR2_armeb
>
> Actually, BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS is BR2_i386 has this condition
> more
>
> ((BR2_mips || BR2_mipsel) && !BR2_MIPS_SOFT_FLOAT)
>
> Are you sure LJSYSCALL package works on mips?
>

there is a folder
http://github.com/justincormack/ljsyscall/tree/master/syscall/linux/mips
and the readme (section requirements) contains :
      " On Linux ARM (soft or hard float), x86, AMD64, and PPC32 and MIPS32
(o32 ABI) architectures are supported; this is the complete set of
architectures that LuaJIT supports. "

Fran?ois


> >>         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 &&
> !BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS
> >>
> >> --
> >> 2.1.0
> >>
> >> _______________________________________________
> >> buildroot mailing list
> >> buildroot at busybox.net
> >> http://lists.busybox.net/mailman/listinfo/buildroot
> >
> >
> >
> > --
> > Profile: http://it.linkedin.com/in/compagnucciangelo
>
>
>
> --
> Profile: http://it.linkedin.com/in/compagnucciangelo
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20150318/b229e3a4/attachment.html>

[Buildroot] [PATCH 2/3] ljlinenoise: refactor with BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS

[François Perrad]

2015-03-17 21:32 GMT+01:00 Thomas Petazzoni <
thomas.petazzoni@free-electrons.com>:

> Dear Francois Perrad,
>
> On Tue, 17 Mar 2015 21:04:48 +0100, Francois Perrad wrote:
> > Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
> > ---
> >  package/ljlinenoise/Config.in | 5 ++---
> >  1 file changed, 2 insertions(+), 3 deletions(-)
> >
> > diff --git a/package/ljlinenoise/Config.in
> b/package/ljlinenoise/Config.in
> > index 17be1be..ce1a156 100644
> > --- a/package/ljlinenoise/Config.in
> > +++ b/package/ljlinenoise/Config.in
> > @@ -5,7 +5,7 @@ config BR2_PACKAGE_LJLINENOISE
> >       # 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
> > +     depends on BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS
>
> Are you sure?
>
> BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS is clearly not the same as "BR2_i386
> || BR2_x86_64 || BR2_powerpc || BR2_arm || BR2_armeb". For example,
> mips/mipsel are supported in LuaJIT, but not in the current condition
> of ljlinenoise.
>
>
This package is based on ljsyscall, so see the discussion in [patch 3/3]
ljsyscall

Fran?ois



> Thomas
> --
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20150318/1bd185ba/attachment.html>

[Buildroot] [PATCH 3/3] ljsyscall: refactor with BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS

[Thomas Petazzoni]

Dear Francois Perrad,

On Tue, 17 Mar 2015 21:04:49 +0100, Francois Perrad wrote:

> diff --git a/package/ljsyscall/Config.in b/package/ljsyscall/Config.in
> index dcbd521..b9ff04d 100644
> --- a/package/ljsyscall/Config.in
> +++ b/package/ljsyscall/Config.in
> @@ -3,12 +3,12 @@ config 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
> +	depends on BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS

Since you say yourself in the discussion that the set of architectures
supported by ljsyscall is the same as the set of architectures
supported by LuaJIT, then this 'depends on
BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS' is not needed, because this package
already 'depends on BR2_PACKAGE_LUAJIT'.

So if BR2_PACKAGE_LUAJIT has been enabled,
BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS must be true. So adding this
dependency does not make sense.

Can you rework the ljlinenoise and ljsyscall patches to remove this
architecture dependency?

Thanks,

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