Re: [PATCH] fix for review

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 
*l*ibluajit-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 toDepends: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?