Commit 5b2defa5 authored by Doron Behar's avatar Doron Behar
Browse files

slurm: substituteInPlace with --replace-fail 

Safer, for it will raise an error if substitution will fail.
parent f082c56f

pkgs/by-name/sl/slurm/package.nix

+4 −4
Original line number Diff line number Diff line
@@ -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)' \

        --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'