Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 682990

Summary: games-action/minetest should depend on =dev-lang/luajit-2.1.0_beta3 to avoid crashes in some cases
Product: Gentoo Linux Reporter: Red <redblade7>
Component: Current packagesAssignee: William Breathitt Gray <vilhelm.gray>
Status: UNCONFIRMED ---    
Severity: normal CC: jstein, proxy-maint, redblade7
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
See Also: https://bugs.gentoo.org/show_bug.cgi?id=679730
https://bugs.gentoo.org/show_bug.cgi?id=678404
https://bugs.gentoo.org/show_bug.cgi?id=685010
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on: 685010    
Bug Blocks:    

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.