Time  Nick            Message
00:45 MTDiscord       <luatic> alright #16036 should be ready for review now. thanks for poking me.
00:45 ShadowBot       https://github.com/luanti-org/luanti/issues/16036 -- Safe `require` for module loading by appgurueu
04:52 technomancy     luatic: is it deliberate that you're omitting package.path?
04:54 technomancy     like one alternative would be to construct package.path out of all the various mods and then have the package.loaders entry search that path; would be a little more consistent with regular lua
04:56 technomancy     I also think it would be worth supporting package.preload; this is really nice for bundling up several files into one .lua
04:56 technomancy     and I have a few ideas for more tests to cover other cases
04:56 technomancy     luatic: would you like me to open a PR against your feat/require branch? I could also wait till it's merged to the main repo and open a PR there
13:16 MinetestBot     02[git] 04red-001 -> 03luanti-org/luanti: Restrict (SS)CSM access to `collectgarbage` 1374e03c6 https://github.com/luanti-org/luanti/commit/74e03c60b3d863deeb9d21871abdd27cc0d5c1ef (152026-08-18T13:15:38Z)
13:16 MinetestBot     02[git] 04cx384 -> 03luanti-org/luanti: Don't open chatcommand help window on failed `/all` command (#17424) 130fb103b https://github.com/luanti-org/luanti/commit/0fb103bda7550e5002ef266fa0b7007aedaa3914 (152026-08-18T13:16:02Z)
13:16 MinetestBot     02[git] 04sfan5 -> 03luanti-org/luanti: Clarify security policy (#17459) 139ffb7b6 https://github.com/luanti-org/luanti/commit/9ffb7b6999ebe906b841c2d870156a1774883ff6 (152026-08-18T13:16:19Z)
13:19 MinetestBot     02[git] 04sfan5 -> 03luanti-org/luanti: Harden filesystem sandboxing for main menu 1302d5783 https://github.com/luanti-org/luanti/commit/02d57830a5b30912a4f53dec0643d832ec8fa3fe (152026-08-18T13:18:12Z)
13:19 MinetestBot     02[git] 04sfan5 -> 03luanti-org/luanti: Improve ScriptApiSecurity logging 1310fcbd9 https://github.com/luanti-org/luanti/commit/10fcbd9401ed1b7438be543db45a35496987dd87 (152026-08-18T13:18:18Z)
13:19 MinetestBot     02[git] 04sfan5 -> 03luanti-org/luanti: Fix mishandling of case-insensitive paths 133351ccf https://github.com/luanti-org/luanti/commit/3351ccffdf8db6d2852be8f77e2c56517f09f2e5 (152026-08-18T13:18:25Z)
13:19 MinetestBot     02[git] 04sfan5 -> 03luanti-org/luanti: Prevent Lua access to stdin/stdout entirely 1358b01e9 https://github.com/luanti-org/luanti/commit/58b01e9373add0ca1a8331e1dfb65d3d59a966ae (152026-08-18T13:18:29Z)
15:39 MTDiscord       <luatic> technomancy: yes. i think a custom loader is more natural than package.path because it lets me directly use core.get_modpath. the path alternative would be messier (i'd have to set up all the search paths for where mods can be - mod folder, modpacks, game, world - in the correct order).
15:40 MTDiscord       <luatic> i think i would prefer to get this in as-is first, because i think that's already useful (and already a bit tricky in some aspects); non-essential features can be added later
15:41 technomancy     luatic: sorry, I was unclear; I meant that your custom loader could itself use package.path
15:41 technomancy     but now that you mention core.get_modpath I see why this might be preferred
15:42 technomancy     sounds good; I'll do some fiddling around and testing and then wait for the merge before submitting
15:43 MTDiscord       <luatic> thanks!
20:46 swift110-mobile ok cool
23:01 MinetestBot     02[git] 04sfan5 -> 03luanti-org/luanti: Fix testPathsEqual truth table 1388d9032 https://github.com/luanti-org/luanti/commit/88d9032a8070baf09ec59dad7cd5072b2a9608bf (152026-08-18T21:43:59Z)