Time  Nick      Message
01:06 * sofar   rebases #6804
01:06 ShadowBot https://github.com/minetest/minetest/issues/6804 -- Punchwear by sofar
01:08 sofar     huh, getting compile errors now
01:09 sofar     src/script/common/c_converter.cpp:386:11: error: ‘lua_objlen’ was not declared in this scope; did you mean ‘lua_len’?
01:09 sofar     386 |   int n = lua_objlen(L, index);
01:16 sofar     maybe I'm missing luajit includes now
01:28 * sofar   throws hands up in the air... patch by hybriddig broke the rebase and my memory doesn't go far enough to comprehend
01:36 paramat   thanks anyway =
01:36 paramat   )
09:24 p_gimeno  ~tell mtda es2_info comes in package mesa-utils-extra
09:24 ShadowBot p_gimeno: O.K.
09:27 p_gimeno  about getmetatable and sandboxing, the reasons stated in http://lua-users.org/wiki/SandBoxes do not justify disallowing getmetatable, you can save the string metatable before entering the sandbox and restore it afterwards. Mesecons Luacontroller does just that.
09:33 p_gimeno  https://github.com/minetest-mods/mesecons/blob/8baa789eb180c5a8944682c893478d7f9f874970/mesecons_luacontroller/init.lua#L600
09:37 p_gimeno  it gets tricky when it lets Lua functions run within the sandbox, though - https://github.com/minetest-mods/mesecons/blob/8baa789eb180c5a8944682c893478d7f9f874970/mesecons_luacontroller/init.lua#L200
09:39 p_gimeno  a possible solution is to use a getmetatable replacement that detects if type(arg)=="string"
11:52 Krock     !tell paramat That error is not in relation to the PR. https://github.com/minetest/minetest/pull/6804#issuecomment-520193877
11:52 ShadowBot Krock: O.K.
12:57 SophyGUO  Hi all, I am a master student based in the University of Leeds (UK). I am working on a research investigating the relationship between OSS community culture, governance structure, etc. and innovation capability of the community.
12:58 SophyGUO  I found Minetest community is active and supportive. could you please help fill in a questionnaire?
12:59 SophyGUO  the link is https://leedsubs.eu.qualtrics.com/jfe/form/SV_cUPSObEbxQqEX1b
12:59 SophyGUO  this should take you 5-10 minutes to complete. Many thanks!
13:08 Krock     I feel like the topic sounds much more complicated than it actually should be
13:08 Krock     SophyGUO: you'll also have better chances for answers by posting that survey on the forums
13:11 Krock     also the non-english pages should either be fixed or removed. the placeholders look weird
13:12 SophyGUO  Yes, it's a complicated topic, so more data is needed. already posted that on the forum as well.
13:13 SophyGUO  I only posted English version, I will check what goes wrong.
13:14 SophyGUO  Thanks a lot for your advice!
13:16 Krock     np
14:21 Krock     +1 game#2437 will merge in 5 minutes
14:21 ShadowBot https://github.com/minetest/minetest_game/issues/2437 -- furnace: Fix "output full" infotext by pauloue
14:27 Krock     mergign
18:18 Krock     CGUITTFont is a hard piece to understand oof
21:46 paramat   ok thanks