Time Nick Message 01:33 rubenwardy I did a partial roll out, the error rate has increased by 3300% 01:33 rubenwardy of the Android app 01:33 rubenwardy sample size is currently 715 01:34 rubenwardy which is less than 4%, although I have it set to 10% roll out 08:15 sfan5 ? 10:56 Baytuch Good evening, MineTesters 10:59 erlehmann it's 11:58 in the morning 13:27 MTDiscord Hello Baytuch. 14:10 erlehmann more evidence that ingame texture test nodes might really help with debugging IMO https://forum.minetest.net/viewtopic.php?f=6&t=27762&p=406338#p406338 14:10 erlehmann > Tools inside a black box in 5.5.0? 14:10 erlehmann it would be really useful to tell the player to use devtest here 14:11 erlehmann in particular, a unit test would *not* help to figure this out 15:55 sfan5 just found out that all non-curl builds crash when creating a server 15:55 rubenwardy easy fix, make cURL required 15:55 rubenwardy joking, but also semi serious 15:56 rubenwardy would reduce the number of possible configurations 15:59 erlehmann this should be easily bisectable though? 16:24 sfan5 by "crash" I meant a Lua error 16:24 sfan5 so it's already immediately obvious which change it was 16:28 MTDiscord Is curl required for all HTTP stuff, like the Content browser, HTTP API for mods, and server announce? 16:28 MTDiscord Something that pervasively used could very justifiably made required, even if some users really don't intend to use any of those features, because of the amount of surface area involved in making them all optional. 16:32 sfan5 it is 16:35 MTDiscord sfan5, could you take a look at #12026 when you have time? If you like my proposed solution, I am ready to PR it. 16:35 ShadowBot https://github.com/minetest/minetest/issues/12026 -- Throw fatal error if enabled database backends are missing 18:39 sfan5 I think a tri-state variable (TRUE|FALSE|AUTO) would be better 19:02 MTDiscord Maybe (ENABLE|DISABLE|AUTO)? AUTO isn't a valid bool type, so it would be confusing. 19:04 sfan5 the options should be backwards compatible anyway 19:04 sfan5 so ENABLE, ON, TRUE and 1 have to be considered the same 19:04 rubenwardy I don't think there's a problem with true/false 19:05 MTDiscord I found an example on a mailing list using AUTO with an option that looks clean. Thanks for the suggestion. 19:38 MTDiscord YES|NO|SHRUG 19:50 MTDiscord Done. It shall be SHRUG. 19:54 MTDiscord It looks like minetest.get_craft_result() doesn't return multiple craft results if you give it more than enough of the required items? 19:55 MTDiscord My lavastuff mod uses get_craft_result() when digging nodes with cookable drops. Clay blocks drop 4 clay lumps, but when you dig them with lavastuff items they only give one brick 19:55 MTDiscord https://github.com/minetest-mods/lavastuff/blob/4cef3ba46f3e96b80e7f259126163d6d942904c2/init.lua#L115-L120 19:57 MTDiscord I think the builtin recipe system only works with stacks of 1...? 19:57 MTDiscord Ah, it gives you what's leftover instead of resulting it all 19:57 MTDiscord So if you have 4 clay lumps you'd have to search for a recipe using 1 clay lump and then multiply the output by 4 21:14 MTDiscord a local man is causally crashing the engine again