[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 at 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 at 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-0002.html>