| 16 Oct 2025 |
 matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | Still nobody to review my PRs that I'm holding as patches in my own repo | 15:43:59 |
| matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | I've tested them pretty extensively, they're working, and they just live in https://github.com/MatthewCroughan/nixos-musl/blob/refactor/musl.nix#L80-L119 | 15:44:55 |
 dramforever | nice | 15:56:33 |
| dramforever | if it looks like nothing else has broke i'll send it to musl | 15:56:45 |
| dramforever | or maybe think harder about how to deal with arch specific code | 15:56:58 |
| matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | Just make it crash with -95 on every platform except for aarch64 😎 | 15:58:18 |
| dramforever | i already made it fall back to old behavior for unimplemented architectures | 15:59:00 |
| matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | That is an ouch... | 15:59:34 |
| matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | It should at least report or mark that it is broken | 15:59:41 |
| matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | otherwise that's just some form of silent bug that breaks other arches | 15:59:49 |
| matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | I think there should be some messaging somewhere | 16:00:03 |
| matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | Can we carry this musl patch in nixpkgs until they merge it? | 16:00:17 |
| dramforever | no | 16:00:37 |
| matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | I think this is equivalent to how we patch many things, like kmod or systemd because we need recursion | 16:00:38 |
| matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | * I think this is equivalent to how we patch many things, like kmod or systemd because we need recursion and lists | 16:00:43 |
| matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | I don't see how this patch is any more or less relevant than some of the kmod or systemd patches we carry | 16:00:58 |
| matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | BTW, if you feel like more hacking, kmod needs to be version bumped, and that patch is crazy | 16:01:17 |
| matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | https://github.com/NixOS/nixpkgs/blob/nixos-unstable/pkgs/os-specific/linux/kmod/module-dir.patch | 16:01:33 |
| matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | This makes kmod accept a list of arguments instead of a hardcoded fhs dir | 16:02:04 |
| matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | necessary for nixos, similar to your musl patch :) | 16:02:11 |
| matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | And we're 3 versions behind on kmod because they've moved to meson, and this patch isn't upstreamed | 16:02:55 |
| dramforever | vendoring my musl patch in nixpkgs is a violation of https://do-not-ship-work-in-progress.gitlab.io/ | 16:03:59 |
| matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | Is it any more or less WIP than the kmod patch? | 16:04:31 |
| dramforever | the kmod module-dir.patch doesn't apply because it's nixos specific | 16:04:34 |
| dramforever | so we can't not ship it in nixpkgs | 16:04:44 |
| matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | i would argue that the musl patch is nixos specific too, at least I'm sure when you upstream it someone might say something like that | 16:04:54 |
| matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | It's a bit of a blurry line | 16:05:12 |
| matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | I mean, I wonder how many other edge cases are encountered in musl in nixpkgs because of the lack of the patch you just made | 16:05:51 |
| matthewcroughan @ 39c3 (DECT 94667 or 97340 or 67192) | I'm sure it shows up in other places | 16:06:03 |
| dramforever | when upstream says they don't want it then we'll talk | 16:06:28 |
