[Buildroot] [PATCH 01/11] lua: add 5.4.0 version

[Thomas Petazzoni]

Hello François,

Most issues are small details, except the last one, which really needs
a reply from you.

On Thu,  2 Jul 2020 22:22:20 +0200
Francois Perrad <fperrad at gmail.com> wrote:

> diff --git a/package/lua/lua.mk b/package/lua/lua.mk
> index ce75af6b0..0b6d2c1dc 100644
> --- a/package/lua/lua.mk
> +++ b/package/lua/lua.mk
> @@ -4,15 +4,19 @@
>  #
>  ################################################################################
>  
> +ifeq ($(BR2_PACKAGE_LUA_5_4),y)
> +LUA_VERSION = 5.4.0
> +else
>  ifeq ($(BR2_PACKAGE_LUA_5_3),y)

else ifeq

on one line...

>  LUA_VERSION = 5.3.5
>  else
>  LUA_VERSION = 5.1.5
>  endif
> +endif

... as it avoids the need for that additional endif.

>  LUA_SITE = http://www.lua.org/ftp
>  LUA_INSTALL_STAGING = YES
>  LUA_LICENSE = MIT
> -ifeq ($(BR2_PACKAGE_LUA_5_3),y)
> +ifeq ($(BR2_PACKAGE_LUA_5_3)$(BR2_PACKAGE_LUA_5_4),y)
>  LUA_LICENSE_FILES = doc/readme.html
>  else
>  LUA_LICENSE_FILES = COPYRIGHT
> @@ -22,6 +26,10 @@ LUA_PROVIDES = luainterpreter
>  
>  LUA_CFLAGS = -Wall -fPIC -DLUA_USE_POSIX
>  
> +ifeq ($(BR2_PACKAGE_LUA_5_4),y)
> +LUA_CFLAGS += -DLUA_COMPAT_5_3
> +endif
> +
>  ifeq ($(BR2_PACKAGE_LUA_5_3),y)

This could be changed to:

else ifeq ($(BR2_PACKAGE_LUA_5_3),y)

> diff --git a/package/pkg-luarocks.mk b/package/pkg-luarocks.mk
> index 603c0851b..be85563ba 100644
> --- a/package/pkg-luarocks.mk
> +++ b/package/pkg-luarocks.mk
> @@ -20,9 +20,9 @@
>  LUAROCKS_RUN_CMD = $(HOST_DIR)/bin/luarocks
>  LUAROCKS_CFLAGS = $(TARGET_CFLAGS) -fPIC
>  HOST_LUAROCKS_CFLAGS = $(HOST_CFLAGS) -fPIC
> -ifeq ($(BR2_PACKAGE_LUA_5_3),y)
> -LUAROCKS_CFLAGS += -DLUA_COMPAT_5_2
> -HOST_LUAROCKS_CFLAGS += -DLUA_COMPAT_5_2

I don't understand why this is being dropped. Could you clarify ?

> +ifeq ($(BR2_PACKAGE_LUA_5_4),y)
> +LUAROCKS_CFLAGS += -DLUA_COMPAT_5_3
> +HOST_LUAROCKS_CFLAGS += -DLUA_COMPAT_5_3
>  endif

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com