| 18 Oct 2025 |
matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | * Like i'm not letting things hang around unsubmitted | 12:32:51 |
Alyssa Ross | you were asking about mesa/valgrind in a way that suggested urgency | 12:33:09 |
matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | Maybe I'm just excited? :D | 12:33:19 |
matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | Not urgent at all | 12:33:23 |
Alyssa Ross | if there isn't urgency, the thing to do is finish fixing valgrind | 12:33:24 |
dramforever | you seem very vehemently against the idea of nixpkgs just not disabling valgrind for mesa on musl+llvm | 12:33:56 |
matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | No, I just think it's probably the right thing to do, like who needs valgrind on mesa? | 12:34:25 |
dramforever | this is the "urgency" i've been talking about, you've been urging this for like twenty minutes | 12:34:31 |
matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | Like, it's for development and debugging, no? | 12:34:37 |
matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | and it increases the closure size quite a bit probably anyway | 12:34:54 |
matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | Ah yeah it also depends on perl | 12:35:33 |
matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | So can't have a perlless system with this in :D | 12:35:46 |
dramforever | it is for development and debugging, which a lot of nixpkgs users do | 12:35:50 |
Alyssa Ross | if you want to argue for disabling valgrind by default on mesa, that's very different from arguing for doing it on "some platforms". | 12:36:25 |
Alyssa Ross | but presumably at some point somebody decided that Valgrind should be included in Mesa, so it's not necessarily going to be easy to find consensus for | 12:37:26 |
matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | It's optional via override so it's fine | 12:37:46 |
matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | Like I think if it's optional via override, then it doesn't need to touch nixpkgs | 12:38:23 |
matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | And if it's not optional via override like this is, then that's the thing that needs to change. But things should also compile with musl without override, which in this case would require fixing valgrind upstream | 12:39:12 |
Alyssa Ross | again, mesa works fine with musl | 12:39:26 |
matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | s/musl/llvm/g | 12:39:40 |
matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | I just woke up :D | 12:39:50 |
matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | Crawled to my computer and started building | 12:40:02 |
Alyssa Ross | support for building with LLVM in Nixpkgs is very new. musl has had a lot of work put into it over years, and LLVM hasn't yet. | 12:40:12 |
Alyssa Ross | (speaking from experience) it's also very important when doing stuff with obscure platforms to make sure you're not stepping on anybody's toes, because if people get annoyed they'll start wanting to just declare your platform unsupported, and then you're screwed | 12:40:23 |
Alyssa Ross | the #1 most important thing for musl support in Nixpkgs being where it is today (where I don't have to have a fork) is that it happened quietly, so nobody ever got annoyed enough to ask why they should have to care about it | 12:41:56 |
matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | Who merged the PRs and reviewed the PRs though? | 12:42:31 |
Alyssa Ross | package maintainers mostly | 12:42:54 |
Alyssa Ross | I always tried to make very sure that PRs were merge-ready when I opened them | 12:43:08 |
Alyssa Ross | in many cases, there were no Nixpkgs PRs | 12:43:40 |
Alyssa Ross | because I got things fixed upstream | 12:43:44 |