From 6d8cd446c5c7cba12b82bf1f297e86bfa9104156 Mon Sep 17 00:00:00 2001 From: Hector <10837193+he-b@users.noreply.github.com> Date: Tue, 10 Mar 2020 20:41:00 -0500 Subject: [PATCH] Add valgrind variant to petsc (#15427) * Add PETSc valgrind dependency * Remove +debug on valgrind dependency * enh variant description --- var/spack/repos/builtin/packages/petsc/package.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/var/spack/repos/builtin/packages/petsc/package.py b/var/spack/repos/builtin/packages/petsc/package.py index 88332caec98..ddee34e1872 100644 --- a/var/spack/repos/builtin/packages/petsc/package.py +++ b/var/spack/repos/builtin/packages/petsc/package.py @@ -99,6 +99,8 @@ class Petsc(Package): description='Activate X support') variant('batch', default=False, description='Enable when mpiexec is not available to run binaries') + variant('valgrind', default=False, + description='Enable Valgrind Client Request mechanism') # 3.8.0 has a build issue with MKL - so list this conflict explicitly conflicts('^intel-mkl', when='@3.8.0') @@ -146,6 +148,7 @@ class Petsc(Package): depends_on('hdf5+mpi+hl+fortran', when='+hdf5+mpi') depends_on('zlib', when='+hdf5') depends_on('parmetis', when='+metis+mpi') + depends_on('valgrind', when='+valgrind') # Hypre does not support complex numbers. # Also PETSc prefer to build it without internal superlu, likely due to # conflict in headers see @@ -280,7 +283,7 @@ def install(self, spec, prefix): # Activates library support if needed for library in ('metis', 'hdf5', 'hypre', 'parmetis', - 'mumps', 'trilinos', 'fftw'): + 'mumps', 'trilinos', 'fftw', 'valgrind'): options.append( '--with-{library}={value}'.format( library=library, value=('1' if library in spec else '0'))