Time  Nick       Message
10:08 Krock      o/
10:09 Krock      VanessaE: so it's fixed in 5.0.0?
10:09 VanessaE   idk.
10:09 VanessaE   I don't use 5.x
10:09 Krock      that would help to find out whether a fix already exists
10:09 VanessaE   I would have thought someone would have mentioned it in my issue by now....
10:16 VanessaE   I'll give it a try though.
10:22 VanessaE   yep, still present in 5.x - both wrong param2 on place, and getting a visually identical item on dig.
10:45 Krock      okay, thanks for testing
10:46 VanessaE   there, I updated my testing.
10:46 VanessaE   https://github.com/minetest/minetest/issues/7675#issuecomment-431569604
10:46 Krock      I hope that will help finding & fixing the issue. I'm not a specialist on this topic, sadly
10:47 VanessaE   paramat: um...unconfirmed?  *growls loudly*  :(
10:54 paramat    sorry, i should have clarified, because "I'm not sure if it's an engine bug or mod bug"
10:55 jas_       testing should confirm the bug.  where the issue resides is another matter.
10:55 paramat    so 'unconfirmed as an engine bug'
10:55 VanessaE   I'm reasonably sure it's an engine bug wrt. inheriting colors/itemstack meta.
10:56 paramat    a mod error is not a valid engine bug
10:56 paramat    ok will change it back
10:57 VanessaE   thank you.
10:57 paramat    and yes i also avoid hardware colouring stuff, sorry
10:58 VanessaE   thankfully, in practice it's not much more than an annoyance, but it could still cause problems for players (especially if their inventory is full, even if the user has a small stack of the node in question)
11:11 VanessaE   on a different note:  what's the rationale behind making clear_craft() error-out/crash the server if the specified recipe isn't found?  wouldn't a simple warning be better?
11:16 paramat    there usually isn't a rationale :) search for and/or open an issue?
11:18 VanessaE   meh
11:22 paramat    no fix then :)
11:23 Krock      paramat: if param2 placement fails it's an engine bug
11:23 Krock      since it's the engine which implements the param2 coloring feature
11:26 paramat    ok. i didn't really  think about the issue enough to realise :) afk for noms
11:26 VanessaE   paramat: https://github.com/minetest/minetest/issues/6513
11:27 VanessaE   THAT is why I don't file issues much anymore.
11:30 Krock      paramat: FYI: currently all advanced settings dialogues have exactly the same size; but could be changed on demand
11:32 Krock      my bad. they really have different heights since I didn't adjust the layout by size, but rather so that it's compact as possible
12:36 paramat    VanessaE 6513 seems a good idea to me
12:37 VanessaE   paramat: it is.  it's the replies to that issue that I was referring to.
12:37 VanessaE   ("it is" -> "returning false is")
12:43 paramat    sfan's reply is not necessarily implying nothing needs doing, it might just be an observation on the workaround
12:44 VanessaE   yes.  offer a workaround instead of a fix?  ahem.
12:46 VanessaE   maybe I'm just being crabby, I dunno.  just my usual gripe about misdirection of development efforts (which has only gotten worse as available manpower has clearly fallen-off lately)
12:48 paramat    nevermind single bad responses (if it was, dunno). i might try to attend to it :)
12:53 * VanessaE wonders what paramat's noms were ;)
12:56 paramat    Krock found a crash you might be able to help with #7803
12:56 ShadowBot  https://github.com/minetest/minetest/issues/7803 -- MT crashes when using 'restore defaults' for font_size in advanced settings
13:08 Krock      freezes here
15:35 Fixer      Krock: in game?
16:06 Krock      Fixer: no, when resetting
16:07 Krock      mutex lock issue
16:25 Krock      Will merge #7802 in 20 minutes
16:25 ShadowBot  https://github.com/minetest/minetest/issues/7802 -- Mainmenu: Clean up and improve advanced settings dialogues by SmallJoker
17:14 Krock      s/20/50/
17:14 Krock      merging...