Bug 682990 – games-action/minetest should depend on =dev-lang/luajit-2.1.0_beta3 to avoid crashes in some cases

• Product: Gentoo Linux
• Status: RESOLVED WONTFIX

[Description Red 2019-04-09 23:51:20 UTC]

https://forum.minetest.net/viewtopic.php?f=3&t=18263

LuaJIT v2.1.0-beta3 is required if Minetest uses more than 2 GB RAM, which happens if you use a lot of mods with the game (either when using single player or when running a dedicated server for players to connect to). If this happens and a stable LuaJIT version is used, the server crashes.

As this is not an issue in most use cases, I have filed it as "minor"

[Comment 1 Red 2019-04-10 00:12:29 UTC]

It says also at the link, with instructions, that GC64 needs to be enabled in the LuaJIT build for the crashes to be avoided.

Perhaps it could be added as a use flag for =dev-lang/luajit-2.1.0_beta3?

[Comment 2 William Breathitt Gray 2019-04-17 13:31:44 UTC]

(In reply to Red from comment #1)
> It says also at the link, with instructions, that GC64 needs to be enabled
> in the LuaJIT build for the crashes to be avoided.
> 
> Perhaps it could be added as a use flag for =dev-lang/luajit-2.1.0_beta3?

LuaJIT can be compiled with GC64 mode enabled by building with "-DLUAJIT_ENABLE_GC64" set. However, it looks like the current luajit-2.1.0_beta3.ebuild file does not pass that option down. If the luajit ebuild is updated to permit GC64 mode to be enabled, I'll update the minetest ebuild to allow it be selected.

[Comment 3 Red 2019-05-02 21:16:04 UTC]

(In reply to William Breathitt Gray from comment #2)
> (In reply to Red from comment #1)
> > It says also at the link, with instructions, that GC64 needs to be enabled
> > in the LuaJIT build for the crashes to be avoided.
> > 
> > Perhaps it could be added as a use flag for =dev-lang/luajit-2.1.0_beta3?
> 
> LuaJIT can be compiled with GC64 mode enabled by building with
> "-DLUAJIT_ENABLE_GC64" set. However, it looks like the current
> luajit-2.1.0_beta3.ebuild file does not pass that option down. If the luajit
> ebuild is updated to permit GC64 mode to be enabled, I'll update the
> minetest ebuild to allow it be selected.

Apologies that I completely forgot to do this. The whole Minetest site crashed a few days ago, they are still working on repairing it, the main site is back, the wiki and forum will take a few more days, they say. Once I can again link to and view the forum thread here, I will file a bug with LuaJIT.

[Comment 4 Red 2019-05-03 22:42:17 UTC]

(In reply to Red from comment #3)
> (In reply to William Breathitt Gray from comment #2)
> > (In reply to Red from comment #1)
> > > It says also at the link, with instructions, that GC64 needs to be enabled
> > > in the LuaJIT build for the crashes to be avoided.
> > > 
> > > Perhaps it could be added as a use flag for =dev-lang/luajit-2.1.0_beta3?
> > 
> > LuaJIT can be compiled with GC64 mode enabled by building with
> > "-DLUAJIT_ENABLE_GC64" set. However, it looks like the current
> > luajit-2.1.0_beta3.ebuild file does not pass that option down. If the luajit
> > ebuild is updated to permit GC64 mode to be enabled, I'll update the
> > minetest ebuild to allow it be selected.
> 
> Apologies that I completely forgot to do this. The whole Minetest site
> crashed a few days ago, they are still working on repairing it, the main
> site is back, the wiki and forum will take a few more days, they say. Once I
> can again link to and view the forum thread here, I will file a bug with
> LuaJIT.

Apologies again for the delay.

See LuaJIT bug 685010.

[Comment 5 Marek Szuba 2021-05-12 12:45:36 UTC]

I really would rather not introduce a dependency on a four year-old beta-grade version of LuaJIT that is now a looong way behind the Git head - and that isn't keyworded for any of the arches supported by games-action/minetest. Maybe they'll finally release something some time soon?