Time  Nick           Message
07:32 appguru        mod trust is fundamentally broken, see https://github.com/minetest/minetest/issues/12182#issuecomment-1212817260
08:09 sfan5          not surprising
09:02 rubenwardy     merging #12591 and #12685 in 10
09:02 ShadowBot      https://github.com/minetest/minetest/issues/12591 -- Physics overrides: Move values to a common struct by SmallJoker
09:02 ShadowBot      https://github.com/minetest/minetest/issues/12685 -- Fix crash when crafting callbacks return strings instead of ItemStacks by Zughy
10:56 appguru        Zughy: Can you add #12353 to the 5.7 milestone? Lars H.'s comment seems to endorse putting it on the milestone.
10:57 ShadowBot      https://github.com/minetest/minetest/issues/12353 -- Fix acceleration by appgurueu
11:02 appguru        thanks :)
11:08 rubenwardy     https://github.com/minetest/minetest.github.io/pull/261
11:50 Zughy[m]       celeron55: https://github.com/minetest/minetest.github.io/issues/200
13:01 celeron55      Zughy[m]: that's been on my to-do list for a very long time and I now tried to fix it
13:01 celeron55      and failed, of course. see the comment there
13:58 MTDiscord      <Jordach> looks good
13:58 MTDiscord      <Jordach> and works fine
14:07 Zughy[m]       for any core dev that wants to close two PRs in one shot, this takes 30 seconds to review #12678
14:07 ShadowBot      https://github.com/minetest/minetest/issues/12678 -- Reassure previous `nil` behaviour for `tiles` and `special_tiles` by Zughy
15:11 nrz            wow, just see the voumetric lighting feature it's amazing, clearly what i wanted 5 years ago while i was more active in core engine ?
15:24 MTDiscord      <Jordach> Sometimes it just happens nrz
15:25 MTDiscord      <Jordach> Arguing the impossible only ensures it’s arrival
16:14 appguru        Zughy: "This is not a regression. The bug already existed before the advanced particle animations." - Krock on #12657
16:14 ShadowBot      https://github.com/minetest/minetest/issues/12657 -- Clients < 5.6 crash/hang when sent new-style particles
16:14 appguru        Consider retagging & renaming appropriately
16:15 appguru        Also the issue author: "EDIT: Last sentence is in error, I was simply supplying too large an animation value (the documentation doesn't make it explicit that this is time in seconds, rather than the number of frames, and I forgot about that ...)"
16:16 appguru        Issue title should be something along the lines of "invalid particles make clients crash/hang"
16:16 sfan5          but they're not invalid
16:18 sfan5          only with the new 5.6 particles do mod authors have a reason to use a negative frame length
16:19 sfan5          whether this is a regression is philosophical but it should be fixed in 5.6.1
22:05 celeron55      here's some wtf i made tonight https://github.com/minetest/minetest/pull/12690
22:47 rubenwardy     nice celeron55! How well is it performing now?
22:48 Zughy[m]       He's made a PR... He's made a PR! KNEEL BEFORE HIM AND HIS 3K LINES, YOU STUPID MORTALS! THEE WHO BEAR NO FAITH, HOW DARE THEE!
22:51 schwarzwald[m] 3k in one night!?
22:51 MTDiscord      <Jordach> he wrote the original code many years ago