[Buildroot] [PATCH 1/4] swupdate: allowed luajit to enable HAVE_LUA

Jordan Yelloz jordan at yelloz.me
Mon Jul 25 16:46:23 UTC 2016


On 07/25/2016 12:31 AM, Thomas Petazzoni wrote:
> Hello,
> 
> On Sun, 24 Jul 2016 16:27:36 -0700, Jordan Yelloz wrote:
> 
>> diff --git a/package/swupdate/swupdate.mk b/package/swupdate/swupdate.mk
>> index 313a54febd87..7a0fcaea5741 100644
>> --- a/package/swupdate/swupdate.mk
>> +++ b/package/swupdate/swupdate.mk
>> @@ -42,6 +42,9 @@ endif
>>  ifeq ($(BR2_PACKAGE_LUA),y)
>>  SWUPDATE_DEPENDENCIES += lua host-pkgconf
>>  SWUPDATE_MAKE_ENV += HAVE_LUA=y
>> +else ifeq ($(BR2_PACKAGE_LUAJIT),y)
>> +SWUPDATE_DEPENDENCIES += luajit host-pkgconf
>> +SWUPDATE_MAKE_ENV += HAVE_LUA=y
> 
> If both luajit and classic lua are supported, then could you instead
> use:
> 
> ifeq ($(BR2_PACKAGE_HAS_LUAINTERPRETER),y)
> SWUPDATE_DEPENDENCIES += luainterpreter host-pkgconf
> SWUPDATE_MAKE_ENV += HAVE_LUA=y
> 
> luainterpreter is a virtual package that automatically depends either
> on lua or luajit, depending on which of the two is enabled in the
> configuration.
> 
> Could you test this solution, and send an updated version of your patch?
> 
> Thanks a lot!
> 
> Thomas
> 

Hi,

I've updated my patch to reflect your changes and it still works. I'll
resubmit the patch series if you're okay with my explanation for patch 3/4.

--
Jordan



More information about the buildroot mailing list