| 18 Oct 2025 |
matthewcroughan | * 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 | Maybe I'm just excited? :D | 12:33:19 |
matthewcroughan | 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 | 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 | Like, it's for development and debugging, no? | 12:34:37 |
matthewcroughan | and it increases the closure size quite a bit probably anyway | 12:34:54 |
matthewcroughan | Ah yeah it also depends on perl | 12:35:33 |
matthewcroughan | 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 | It's optional via override so it's fine | 12:37:46 |
matthewcroughan | Like I think if it's optional via override, then it doesn't need to touch nixpkgs | 12:38:23 |
matthewcroughan | 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 | s/musl/llvm/g | 12:39:40 |
matthewcroughan | I just woke up :D | 12:39:50 |
matthewcroughan | 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 | 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 |