Nix + dotnet | 121 Members | |
| 24 Servers |
| Sender | Message | Time |
|---|---|---|
| 28 Jun 2025 | ||
Good point, I have 64GB and was reproducing the issue fairly regularly. I take it there's no artificial memory limit from
Could be something like that. Or could just be random segfaults in buggy software... 🤔
Anything specific you want me to test, other than just | 20:49:59 | |
| If it's building in the sandbox and under the systemd daemon, it might have different limits than dev shell. The nuclear logging option is probably `strace -f`. That's going to be a lot of output though. Maybe I'll see if I can find anything useful in `VSTestTask` | 21:00:06 | |
| 30 Jun 2025 | ||
| I'm wondering if it is practical to move the tests into a That way 1) end-users wouldn't need to run the tests to build the unfree package, 2) intermittent tests wouldn't cause build failures on hydra. I'd still like to uncover the actual issue, but I'm curious if extracting the tests to a separate derivation is possible? I guess the test derivation would need the same That sounds reasonable, but what I'm unsure of is how easy it'd be to tell | 12:19:21 | |
Looks like the impl is here: https://github.com/dotnet/dotnet/blob/main/src/vstest/src/Microsoft.TestPlatform.Build/Tasks/VSTestTask.cs There's a note about the "Task returned false but did not log an error." error at line 69. Other than that, I nothing is jumping out at me 🤔 Also: I noticed that the | 12:39:42 | |
| It looks like the same errors occasionally shows up in upstream's CI too. E.g. here | 17:35:27 | |
| * It looks like the same (or similar) errors occasionally shows up in upstream's CI too. E.g. here | 17:39:46 | |
| 1 Jul 2025 | ||
In reply to @mattsturg:matrix.org One way to do this would be to disable tests and just put an override with them enabled in Is it consistently crashing in certain tests? Maybe we can get away with disabling only certain tests or projects? | 01:19:19 | |
Good idea. I've opened https://github.com/NixOS/nixpkgs/pull/421459 Overriding is a simple solution, but it means building the app twice... I was hoping we'd be able to reuse the binaries from the main package, as an input for the test package.
I'm not sure; I don't think it prints which test failed in the logs. It might be hidden in the | 10:37:33 | |
| 6 Jul 2025 | ||
| 02:25:27 | ||
| 11 Jul 2025 | ||
| 16:52:34 | ||
| 14 Jul 2025 | ||
| 13:17:33 | ||
Hey, what can i do if i need to overrideAttrs nugetDeps? Has anyone here ever been in this situation? | 16:59:18 | |
| Why? | 17:19:22 | |
| Is it definitely broken? I'd like it to work. | 17:21:19 | |
| Due to this, i mean. Overriding | 17:33:00 | |
| Or any other attribute from that list | 17:34:14 | |
| I guess we either need to pass all these attributes through, or add something like `overridePythonAttrs`. I don't love either of those options | 18:03:47 | |
probably something like overrideDotnetAttrs | 18:04:18 | |
making it works with plain overrideAttrs would require to move any and all logic done in nix in buildDotnetModule over to bash which would make it much worse | 18:05:04 | |
| It doesn't actually have to be in bash, but it needs to use finalAttrs, and stuff would get leaked into bash | 18:24:14 | |
| that feels like it'd end up being a major mess | 18:24:38 | |
a mix of finalAttrs and non-finalAttrs usage or some other type of hack | 18:25:04 | |
By removing line 215 locally, shouldn't I then be able to override nugetDeps properly with overrideAttrs? | 18:30:36 | |
Possibly not, if it's not using finalAttrs.nugetDeps | 18:31:01 | |
| just to see things working | 18:31:04 | |
| no, because what you want to affect is on line 233 | 18:31:15 | |
| oh, i see | 18:31:34 | |
Here's an issue about python moving away from overridePythonAttrs to overrideAttrs: https://github.com/NixOS/nixpkgs/issues/379602 | 18:32:02 | |
it'd only work if it was something like inherit (finalAttrs) nugetDeps; | 18:32:02 | |
I see, I didn't know overridePythonAttrs was discouraged | 18:32:45 | |