Re: [PATCH] fix for review

[tshah <tshah@linux.ibm.com>]

Hello Dima, 

Thank you for reviewing the patch.

yes, you are right the resulting package dependency changes to Depends:liblua5.3-0. This change was necessary because libluajit-5.1-dev does not support ppc64el and since luaL_setfuncs is only available in Lua ≥5.2.

Lua 5.3 is the default version of Lua in Debian for general-purpose use and is well-supported across all release architectures including ppc64el.

As per my knowledge, there is no major downside of using this version unless:

• The package doesnt use Lua 5.1-specific features ( e.g., setfenv, getfenv) that were either changed/removed in the later versions.
• It is not depending on binary Lua modules built against Lua 5.1 ABI
• And upstream doesn’t explicitly lock to Lua 5.1.

    And, sysdig doesnt follow any of the above as of now.

The only one possible situation where I think lua5.1 will be required specifically but not lua5.3 is for the packages depending on LuaJIT, which is compatible with Lua 5.1 but not with Lua 5.3.

On 21/08/25 11:56, Dima Kogan wrote:

> I see that this changes the resulting package to Depends:liblua5.3-0.
> I'm not super familiar with the lua ecosystem. Does this have downsides?
> Is there some situation where lua5.1 is available and usable, but lua5.3
> isn't?
>