Re: [OE-core] [PATCH V3] lua: upgrade from 5.4.8 to 5.5.0

[Changqing Li <changqing.li@windriver.com>]

[-- Attachment #1: Type: text/plain, Size: 2733 bytes --]


On 3/3/26 23:16, Ross Burton wrote:
> CAUTION: This email comes from a non Wind River email account!
> Do not click links or open attachments unless you recognize the sender and know the content is safe.
>
> Hi,
>
>> On 27 Feb 2026, at 07:08, Changqing Li via lists.openembedded.org<changqing.li=windriver.com@lists.openembedded.org> wrote:
>> Refer [1], PLATS linux-readline is moved, and readline is changed to
>> load dynamically, keep readline as PACKAGECONFIG to allow user to remove
>> readline dependency.
> As is traditional with people using dlopen(), this is now broken…
>
> Lua defaults to dlopen() support being enabled (LUA_USE_DLOPEN is defined on linux) and the readline library is set to libreadline.so<http://libreadline.so/>.
>
> So the PACKAGECONFIG is not doing the right thing anymore: having readline in the sysroot doesn’t change the build, and has no impact on the runtime packages.
>
> The obvious answer is to change the PACKAGECONFIG to “,,,readline” so that there’s still an option that has the effect of enabling readline in lua, but that also won’t work as the on-disk file isn’t called libreadline.so<http://libreadline.so/> unless you install the -dev package…. This means we should also set LUA_READLINELIB to "libreadline.so.8”.
>
> Bonus points for adding readline as a build dependency and looking up what the libreadline.so<http://libreadline.so/> symlink resolves to, but I don’t expect the readline soname to change in the near future so this can be considered overkill.

Hi, Ross

Thanks for pointing out this.  How about we just install -dev packages? 
so that we don't need a local patch to set LUA_READLINELIB to 
"libreadline.so.8”. And

in this way, we are align with lua upstream expected (if readline-dev 
installed, readline is used, otherwise, not use readline)

Here is the diff based on V3:

  PACKAGECONFIG ??= "readline"
-PACKAGECONFIG[readline] = ",,readline"
+PACKAGECONFIG[readline] = ",,,readline-dev"

  TARGET_CC_ARCH += " -fPIC ${LDFLAGS}"
  EXTRA_OEMAKE = "'CC=${CC} -fPIC' 'MYCFLAGS=${CFLAGS} -fPIC' 
MYLDFLAGS='${LDFLAGS}' 'AR=ar rcD' 'RANLIB=ranlib -D'"
@@ -64,3 +64,5 @@ BBCLASSEXTEND = "native nativesdk"

  inherit multilib_script
  MULTILIB_SCRIPTS = "${PN}-dev:${includedir}/luaconf.h"
+
+INSANE_SKIP:${PN} += "dev-deps"

With V3 patch:

root@qemux86-64:~# lua -i -W
Lua 5.5.0  Copyright (C) 1994-2025 Lua.org, PUC-Rio
Lua warning: library 'libreadline.so' not found
 >

With the V4 patch:

root@qemux86-64:/usr/lib# lua -i -W
Lua 5.5.0  Copyright (C) 1994-2025 Lua.org, PUC-Rio
 >

if you think this is ok,  I will send a V4 like this.

Regards

Changqing

>
> Ross

[-- Attachment #2: Type: text/html, Size: 4308 bytes --]