Time  Nick       Message
00:22 p_gimeno   Sorta off-topic, but the acquisition of GH by MS was completed on Oct 26. https://blogs.microsoft.com/blog/2018/10/26/microsoft-completes-github-acquisition/
14:35 rubenwardy pushing trivial bugfix in 10: https://github.com/rubenwardy/minetest/commit/81d55338fa26cf26d43e1e632b9a089a787230dd
14:46 rubenwardy fonr
14:46 rubenwardy *done
14:46 crazyR     \o/ thank you :D#
14:47 rubenwardy :D
14:47 rubenwardy as a general rule, "crash when calling Lua function at load time" is always due to an uninitialised m_env
14:47 rubenwardy this has happened a few times before
14:48 rubenwardy https://github.com/minetest/minetest/commit/a953ff4dfc9f786ee379643b6b498a5699203ff1
14:50 rubenwardy would be nice to catch this with linter
14:50 rubenwardy a @Nullable decoration for pointers would be cool
15:01 T4im       wouldn't you go for a reference anyway if you didn't want it non-nullable?
15:02 T4im       did*
15:02 * T4im     stumbles over notunder-negating
15:03 T4im       if it happens that frequently, maybe creating some kind of NullObject might be an idea?
15:04 rubenwardy references don't support polymorphism
15:05 rubenwardy so it would be a smart pointer of some kind
15:05 rubenwardy which, btw, Minetest should start using more
15:05 T4im       :D
18:14 Krock      will merge #7856 in 10 minutes
18:14 ShadowBot  https://github.com/minetest/minetest/issues/7856 -- Set subtle fall bobbing as the default instead of none by HybridDog
18:15 Krock      *angrily shakes fist* Why do these modders always want to use stuff which is unavailable by the time they want it?
18:16 Krock      -> Re-defining nodes in non-on_mods_loaded callbacks
18:16 Krock      -> Getting server status inside uninitialized server
18:34 Krock      ...merging
19:17 crazyR     has anyone ever considered implementing a RCON like protocol into minetest?
19:19 rubenwardy ~g rcon protocol
19:19 ShadowBot  rubenwardy: Source RCON Protocol - Valve Developer Community - The Source RCON Protocol is a TCP/IP-based communication protocol used by Source Dedicated Server, which allows console commands to be issued to the server via a "remote console", or RCON.The most common use of RCON is to allow server owners to control their game servers without direct access to the machine the server is running  (8 more messages)
19:19 Fixer      ah, that one released not long ago by valve?
19:24 T4im       rcon is an older concept already found in older quake based games; in its simplest version you'd simply have a chatcommand with authentication that executes a passed command as if it were entered into the server console
19:24 T4im       assuming the connection is secured as well
19:25 T4im       although iirc the protocol used to handle those packets indeed a bit differently so you could send them outside the game as well
19:33 crazyR     theres actually a C++ library already: https://developer.valvesoftware.com/wiki/Source_RCON_Protocol#C.2B.2B
19:33 crazyR     hmm.....
19:34 crazyR     oh wait my bad
19:35 T4im       right now rcon isn't much useful to minetest i think; you are probably just talking about the option to connect from the outside to it; but otherwise there is no difference between client-local commands, client-transmitted commands and server commands yet
19:36 sfan5      rcon is so simple that we wouldn't need a library to implement it
19:36 T4im       indeed
19:37 crazyR     rcon because useful when you want to use it for automation from say a website. i know theres other ways todo this... but i think a universal "api" would be interesting ... imo
19:37 crazyR     *could
20:19 Jordach    paramat, there's tons of things so  wrong with sky.cpp, it'd burn it down and start anew
20:20 rubenwardy loool
20:22 paramat    that can be applied to most of MT :)
20:48 Jordach    paramat, mind letting me push the fix?
20:48 Jordach    to my PR
21:13 paramat    i don't mind you adding the fix to your PR
21:14 paramat    #7859 trivial will merge in 30 mins if no objections
21:14 ShadowBot  https://github.com/minetest/minetest/issues/7859 -- Night sky: Fix threshold for applying night colours by paramat
21:15 paramat    a visual improvement and solves a little mystery, thanks for finding that bug
22:01 paramat    for 7859 i'll alter the threshold value a little to not be so sensitive to tiny changes in future, otherwise i can see this bug happening again easily
22:25 paramat    done, retested, merging
22:25 paramat    done