From c8045fa0526774a6b1da979baee2f98a7185dbad Mon Sep 17 00:00:00 2001 From: Efraim Flashner Date: Tue, 12 Jul 2022 21:33:27 +0300 Subject: [PATCH] gnu: openmpi: Only build with valgrind on supported systems. * gnu/packages/mpi.scm (openmpi)[inputs]: Wrap valgrind in if-supported macro. [arguments]: Adjust configure-flags based on if valgrind is present. --- gnu/packages/mpi.scm | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/gnu/packages/mpi.scm b/gnu/packages/mpi.scm index 01dc9e355b..ebf9238906 100644 --- a/gnu/packages/mpi.scm +++ b/gnu/packages/mpi.scm @@ -4,7 +4,7 @@ ;;; Copyright © 2014 Ian Denhardt ;;; Copyright © 2016 Andreas Enge ;;; Copyright © 2017 Dave Love -;;; Copyright © 2017 Efraim Flashner +;;; Copyright © 2017, 2022 Efraim Flashner ;;; Copyright © 2018–2022 Tobias Geerinckx-Rice ;;; Copyright © 2018 Paul Garlick ;;; Copyright © 2019, 2021 Ricardo Wurmus @@ -217,17 +217,21 @@ bind processes, and much more.") (if-supported psm) (if-supported psm2) (if-supported ucx) + (if-supported valgrind) (list rdma-core - valgrind slurm)))) ;for PMI support (launching via "srun") (native-inputs (list pkg-config perl)) (outputs '("out" "debug")) (arguments `(#:configure-flags `("--enable-mpi-ext=affinity" ;cr doesn't work - "--enable-memchecker" "--with-sge" - "--with-valgrind" + + ,@(if ,(package? (this-package-input "valgrind")) + `("--enable-memchecker" + "--with-valgrind") + `("--without-valgrind")) + "--with-hwloc=external" "--with-libevent"