| 21 Jan 2025 |
 emily | I use assert …; for that stuff. but I think you'd want to get things wired up before actually expecting people to set it per ^, otherwise it'll be one very big PR | 02:32:39 |
 Tristan Ross | Oh | 02:32:52 |
| Tristan Ross | It wouldn't even take effect until 25.11 | 02:33:36 |
| emily | well, ideally we can get the prep done before the 25.05 release :) | 02:36:59 |
| emily | splitting it up will probably help with that | 02:37:11 |
| Tristan Ross | Yeah | 02:37:22 |
| Tristan Ross | Ok, the "legacy" use* has been wired up to set the new attributes and pkgsLLVM looks to work with that | 02:37:50 |
| Tristan Ross | Oh I've got good news | 03:08:03 |
| Tristan Ross | emily: the seccomp issue is not an issue anymore lol, checks work now | 03:08:21 |
| Tristan Ross | I didn't change anything lol | 03:08:33 |
| Tristan Ross | Upstream must've fixed it | 03:08:42 |
| Tristan Ross | Or it might've been a codegen bug? | 03:09:00 |
| emily | that's one fewer problem at least :) | 03:13:57 |
| Tristan Ross | Yeah lol | 03:14:32 |
| Tristan Ross | Cool pkgs/development/compilers/llvm/common/default.nix no longer depends on useLLVM | 03:18:40 |
| Tristan Ross | Just need to figure out why threadsCross has an optional for !useLLVM | 03:27:48 |
| Tristan Ross | And then we've got the rust mess | 03:27:54 |
| Tristan Ross | I've been able to adjust some other stuff thankfully because of bugs which no longer occur | 03:28:36 |
| Tristan Ross | I bisected the change to John Ericson (https://github.com/NixOS/nixpkgs/pull/73195/files#diff-ab5748dc9567516fefba8344056b51ec1866adeace380f46e58a7af3d619ea22R11302) | 03:31:14 |
| emily | I think that is just inherited from crossLibcStdenv, maybe. | 03:37:05 |
| emily | I think mcfgthreads might be kind of broken though. there's some weird GCC patch stuff that I think no longer applies(?) | 03:37:28 |
| Tristan Ross | So I can just drop that case of useLLVM? | 03:41:18 |
| emily | I don't know. I just mean it looks like Ericson was copying it from crossLibcStdenv which already had the conditional. | 03:41:44 |
| Tristan Ross | Oh | 03:42:23 |
| Tristan Ross | I'll remove it and we'll see what Ericson thinks | 03:43:19 |
| Tristan Ross | Ok, useLLVM isn't specified in the tree any more | 03:45:01 |
| Tristan Ross | useArocc and useZig didn't get used at all | 03:45:36 |
| emily | probably not a good idea to just remove it without testing/trying to figure out why it was there. 5f134ec6cf66b6a13493b28c463d86a06b283402 seems relevant | 03:45:48 |
| Tristan Ross | Ok, I readded it but now it matches for the LLVM specific libs (compiler-rt, libunwind, libcxx) | 03:48:55 |
 John Ericson | Uh oh! | 03:49:07 |
