| 26 Jun 2026 |
Vladimír Čunát | Jan Tojnar is in .maintainers of that test. | 13:05:12 |
K900 | I'll just do a PR to use another font | 13:09:52 |
Vladimír Čunát | I forced the channel update already
567a49d1913c..e73de5be04e0 nixos-unstable
as 10 days lag is way too much.
| 13:11:30 |
jappie | I don't think I've ever seen 9 channels this close together :p | 13:14:53 |
jappie |  Download image.png | 13:14:53 |
Myria | Applying my bump to master doesn't have the issue it seems. Might be a different problem, I'll bisect applying the commits and see if I can isolate it | 13:16:26 |
emily | no longer the default in GNOME even | 13:27:14 |
emily | oh, sorry missed context | 13:27:22 |
emily | still building Darwin stdenv on the big-rebuild PR fwiw | 13:28:04 |
K900 | https://github.com/NixOS/nixpkgs/pull/535675 | 13:41:38 |
Grimmauld (any/all) | I also wonder how dumb of an idea it would be to just unconditionally throw OMP_NUM_THREADS=1 into our stdenv, to fix most of the scheduling nightmares | 13:43:26 |
Grimmauld (any/all) | diff --git a/pkgs/stdenv/generic/make-derivation.nix b/pkgs/stdenv/generic/make-derivation.nix
index 89d413931c1b..be5afd66ad05 100644
--- a/pkgs/stdenv/generic/make-derivation.nix
+++ b/pkgs/stdenv/generic/make-derivation.nix
@@ -928,6 +928,7 @@ let
cmakeFlags ? [ ],
mesonFlags ? [ ],
+ ompNumThreads ? 1,
meta ? { },
passthru ? { },
pos ? # position used in error messages and for meta.position
@@ -961,7 +962,11 @@ let
let
env' =
- if attrs ? meta.mainProgram then env // { NIX_MAIN_PROGRAM = attrs.meta.mainProgram; } else env;
+ env
+ // {
+ OMP_NUM_THREADS = ompNumThreads;
+ }
+ // lib.optionalAttrs (attrs ? meta.mainProgram) { NIX_MAIN_PROGRAM = attrs.meta.mainProgram; };
derivationArg = makeDerivationArgument (
removeAttrs attrs argumentAttrsToRemove
something like this maybe | 13:45:13 |
Grimmauld (any/all) | (+ docs ig) | 13:45:27 |
Grimmauld (any/all) | that seems to work, but i am also somewhat clueless about best practices around stdenv, not sure i should be the one doing that PR... | 13:46:15 |
emily | don't let llakala see | 13:46:24 |
emily | tbh I would prefer a slightly less blunt instrument if possible | 13:46:47 |
emily | at least unless we're going to set a bunch of other stuff in stdenv too | 13:46:55 |
emily | (if we do want to do this it should probably be in the Bash code though) | 13:47:06 |
Grimmauld (any/all) | We'd want to allow overriding it, right? | 13:47:22 |
emily | we have an mpiCheckPhaseHook | 13:47:23 |
emily | I guess stuff is just not using it? | 13:47:26 |
emily | yeah, so := it | 13:47:32 |
emily | what I think I'd prefer is a default preCheck hook that yells if it can tell you're about to do OpenMP stuff but don't have mpiCheckPhaseHook, or something | 13:48:01 |
emily | because it sets a bunch of other stuff too | 13:48:12 |
emily | admittedly "tell you're about to do OpenMP stuff" might be complicated | 13:48:34 |
dramforever | openmp and mpi are almost unrelated | 13:48:47 |
dramforever | might explain why | 13:48:59 |
emily | well | 13:49:06 |
emily | mpiCheckPhaseHook sets OMP_NUM_THREADS | 13:49:11 |
emily | so maybe I'm not the only one confused by this :P | 13:49:23 |