From 5b2defa5808be36166dc88419496501cb609bfb6 Mon Sep 17 00:00:00 2001 From: Doron Behar Date: Mon, 6 Apr 2026 10:17:33 +0300 Subject: [PATCH] slurm: substituteInPlace with --replace-fail Safer, for it will raise an error if substitution will fail. --- pkgs/by-name/sl/slurm/package.nix | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/pkgs/by-name/sl/slurm/package.nix b/pkgs/by-name/sl/slurm/package.nix index ea06a5f93d88..86121e3d22aa 100644 --- a/pkgs/by-name/sl/slurm/package.nix +++ b/pkgs/by-name/sl/slurm/package.nix @@ -62,18 +62,18 @@ stdenv.mkDerivation (finalAttrs: { prePatch = '' substituteInPlace src/common/env.c \ - --replace "/bin/echo" "${lib.getExe' coreutils "echo"}" + --replace-fail "/bin/echo" "${lib.getExe' coreutils "echo"}" # Autoconf does not support split packages for pmix (libs and headers). # Fix the path to the pmix libraries, so dlopen can find it. substituteInPlace src/plugins/mpi/pmix/mpi_pmix.c \ - --replace 'xstrfmtcat(full_path, "%s/", PMIXP_LIBPATH)' \ - 'xstrfmtcat(full_path, "${lib.getLib pmix}/lib/")' + --replace-fail 'xstrfmtcat(full_path, "%s/", PMIXP_LIBPATH)' \ + 'xstrfmtcat(full_path, "${lib.getLib pmix}/lib/")' '' + (lib.optionalString enableX11 '' substituteInPlace src/common/x11_util.c \ - --replace '"/usr/bin/xauth"' '"${lib.getExe xauth}"' + --replace-fail '"/usr/bin/xauth"' '"${lib.getExe xauth}"' ''); # nixos test fails to start slurmd with 'undefined symbol: slurm_job_preempt_mode'