Time Nick Message 01:27 MTDiscord Oh, hurm, just expose the SDL scancode name rather than the keysym name? 01:28 MTDiscord For qwerty users it won't matter, and for everyone else, they won't be surprised 09:51 CoolJar10 Aa 09:59 sfan5 Bb 10:08 MTDiscord Cc 15:05 sfan5 reminder that some bigger PRs need reviews #11545 #11131 #10100 #8679 (though that one was recently discussed) 15:05 ShadowBot https://github.com/minetest/minetest/issues/11545 -- Animated particlespawners by velartrill 15:05 ShadowBot https://github.com/minetest/minetest/issues/11131 -- [MANUAL MERGE] Async environment for mods to do concurrent tasks by sfan5 15:05 ShadowBot https://github.com/minetest/minetest/issues/10100 -- Additional texture modifiers by Treer 15:05 ShadowBot https://github.com/minetest/minetest/issues/8679 -- Add FormSpec key and mouse events by v-rob 15:43 rubenwardy the async PR is high quality 15:43 rubenwardy I like the devtest tests 15:51 sfan5 there should be more tests of engine functionality like this 15:52 rubenwardy yeah definitely 15:57 rubenwardy merging #11876 in 5 15:57 ShadowBot https://github.com/minetest/minetest/issues/11876 -- Update mods_here.txt to mention installing mods via CDB by qwerty123a2 16:19 rubenwardy why are there multiple packers? 16:20 sfan5 what 16:24 rubenwardy ohh it's per userdata/etc 16:25 sfan5 each userdata can have (un)packing callbacks, other types don't need it 16:52 rubenwardy this PR is huge 16:52 rubenwardy checks position half way through. Yay? 17:09 rubenwardy ok done a first pass, will try it out later 18:47 paradust sfan5: I just noticed... https://github.com/minetest/minetest/blob/master/src/terminal_chat_console.cpp#L97 18:48 sfan5 does that no longer work with your changes? 18:48 paradust The LL_TRACE level would now be included in this. That might be the desired result, because previously that would only happen if --trace was also given on the command-line 18:48 paradust It will be more noisy 18:48 sfan5 ah, doesn't matter 18:48 paradust Should I make it do the old behavior you think? 18:48 paradust ah ok 18:48 paradust *be an undesired 18:50 Krock it would be so much easier to offload those particle animations to CSM with basically zero additional operator/function parsing libraries 18:50 Krock (context: animated particle spawners) 19:17 Zughy[m] celeron55: can you look at your Matrix account when you have a chance? 19:18 celeron55 in a moment 19:23 celeron55 ok so, i would like to add Zughy as a triager on github, any objections? 19:24 MTDiscord no objections from me. 19:25 sfan5 sure 19:25 sfan5 as in "no objection" 19:42 celeron55 added 19:49 rubenwardy sure 21:18 erle celeron55 is the role of triager defined somewhere? are there more people than zughy? from what i understand, it is just marking bugs as “confirmed” right? 21:19 Zughy[m] I'm mostly closing "possible close" issues for now, following these guidelines https://dev.minetest.net/Git_Guidelines#Issue_and_Pull-Request_Management 21:21 Zughy[m] I think Wuzzy and wsor are the other ones 21:22 erle well i hope you are careful with that, given that some people use possible close for both “i don't want to deal with this” and “i do not believe this issue exists” (even in the face of evidence) 21:24 erle Zughy[m] can you also remove “possible close”, i.e. from this, once you confirmed it? it's a reproducible crash: https://github.com/minetest/minetest/issues/11742 21:25 Zughy[m] My role is to apply those guidelines, if nobody said anything within one month after putting the label, I just close it 21:25 erle oh lol 21:26 Zughy[m] (which is not the case of that issue) 21:26 erle uh, but it has not been active for over a month. you mean over a month after the label was applied? 21:28 erle because in the latter case it doesn't really matter to me ig 21:29 erle but the thing above is not the only crash that was marked with “possible close” instead of “confirmed” where i think the reason is not wenting to fix the crash 21:29 Zughy[m] yes, I see that the label as a "ok, we're gonna close this unless someone can prove us that this shouldn'be closed" 21:29 erle ah okay 21:29 erle then i guess the amount and quality of comments *after* the possible close matter 21:31 erle i just don't want minetests issue tracker to turn into the kubernetes or mindustry issue trackers. the former closes everything after 90 (?) days of inactivity (even if confirmed), the latter closes everything that can not be confirmed within a very short time period. sorry for being so sceptical, i wish you great success! 21:32 erle Zughy[m], just to confirm: you wouldn't close this one either, right? https://github.com/minetest/minetest/issues/11829 21:32 wsor to clarify, wuzzy and caloun are triagers 21:33 wsor i am not 21:35 wsor *Calinou 21:35 Zughy[m] erle: that's up to core devs to decide, it requires techincal knowledge, I won't touch it. It feels like something to discuss on meetings, but I might be wrong 21:39 erle Zughy[m] if that is your standard of being careful, i am not worried! :) 21:56 MTDiscord Can I get a triager role too? 21:57 rubenwardy I think Triagers should also be able to do unconfirmed -> confirmed if they're able to reproduce it 21:57 MTDiscord ^ 21:57 MTDiscord that's what I would like the role for 21:57 MTDiscord I already raised the issue of too damn many unconfirmed bugs™️ a while ago 22:01 erle i can't promise i would do too much (basically i only want to reproduce), but i have also complained about unconfirmed bugs. i would, of course, never mark my own bugs as confirmed, even if they stay unconfirmed after months despite having clear repro instructions. and i would obv never want to close issues. 22:02 erle so can i have the role too? limited to reproducing things and setting unconfirmed to confirmed only. 22:02 erle (but only if you are okay with me doing 8 bugs in a week and then nothing for 2 or 3 weeks) 22:28 Zughy[m] pointing out that as a triager I can't edit issues titles nor edit people's comments. It would help for issues like #12235 22:28 ShadowBot https://github.com/minetest/minetest/issues/12235 -- I can't open Mindset on my Macbook