Skip to content

Commit

Permalink
mpich: extend the warning about pmix
Browse files Browse the repository at this point in the history
  • Loading branch information
SomeoneSerge committed Jan 12, 2024
1 parent 6cccc07 commit 9476148
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions pkgs/development/libraries/mpich/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,12 @@
# either libfabric or ucx work for ch4backend on linux. On darwin, neither of
# these libraries currently build so this argument is ignored on Darwin.
, ch4backend
# Process manager to build,
# Process managers to build (`--with-pm`),
# cf. https://github.com/pmodels/mpich/blob/b80a6d7c24defe7cdf6c57c52430f8075a0a41d6/README.vin#L562-L586
, withPm ? [ "hydra" "gforker" ]
, pmix
# PMIX support is likely incompatible with process managers (`--with-pm`)
# https://github.com/NixOS/nixpkgs/pull/274804#discussion_r1432601476
, pmixSupport ? false
} :

Expand Down Expand Up @@ -57,8 +59,8 @@ stdenv.mkDerivation rec {
'';

meta = with lib; {
# --with-pmix silently disables gforker
broken = ((builtins.elem "gforker" processManagers) && pmixSupport);
# As far as we know, --with-pmix silently disables all of `--with-pm`
broken = pmixSupport && processManagers != [ ];

description = "Implementation of the Message Passing Interface (MPI) standard";

Expand Down

0 comments on commit 9476148

Please sign in to comment.