[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