This PR migrates geanylua to Lua 5.3 (my oldish distro doesn't provide lua 5.4 so I didn't test, maybe it works too). There were mostly just simple changes. From my limited testing, everything seems to work.
Posting as an (in my opinion better) alternative to #1233 You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany-plugins/pull/1235
-- Commit Summary --
* Require lua 5.3 * Rename luaL_reg to luaL_Reg * Replace luaL_register(L, NULL, funcs) with luaL_setfuncs(L, funcs, 0) * Replace luaL_register() * Rename lua_objlen() to lua_rawlen() * Replace the stack traceback code with the code taken from the LUA binary
-- File Changes --
M build/geanylua.m4 (8) M geanylua/glspi_app.c (4) M geanylua/glspi_dlg.c (6) M geanylua/glspi_doc.c (4) M geanylua/glspi_init.c (4) M geanylua/glspi_kfile.c (8) M geanylua/glspi_run.c (30) M geanylua/glspi_sci.c (6) M geanylua/gsdlg_lua.c (10)
-- Patch Links --
https://github.com/geany/geany-plugins/pull/1235.patch https://github.com/geany/geany-plugins/pull/1235.diff
OK, I just tried on another VM and it compiles and runs with Lua 5.4 too, without any modifications. The only time some modifications were needed was for 5.1->5.2 transition, the 5.2->5.3 transition didn't require any change either.
github-comments@lists.geany.org