From acfa2ea0188129fd1e683fe8e146256b1a695d5c Mon Sep 17 00:00:00 2001 From: Greg Becker Date: Wed, 5 Aug 2020 09:19:22 -0700 Subject: remove hypre variant from mfem and all references to it (#17885) --- var/spack/repos/builtin/packages/axom/package.py | 2 +- var/spack/repos/builtin/packages/cardioid/package.py | 4 ++-- var/spack/repos/builtin/packages/mfem/package.py | 6 ------ 3 files changed, 3 insertions(+), 9 deletions(-) (limited to 'var') diff --git a/var/spack/repos/builtin/packages/axom/package.py b/var/spack/repos/builtin/packages/axom/package.py index add0619a3e..a4e954711a 100644 --- a/var/spack/repos/builtin/packages/axom/package.py +++ b/var/spack/repos/builtin/packages/axom/package.py @@ -124,7 +124,7 @@ class Axom(CMakePackage, CudaPackage): depends_on('umpire cuda_arch={0}'.format(sm_), when='+umpire cuda_arch={0}'.format(sm_)) - depends_on("mfem~mpi~hypre~metis~zlib", when="+mfem") + depends_on("mfem~mpi~metis~zlib", when="+mfem") depends_on("python", when="+python") diff --git a/var/spack/repos/builtin/packages/cardioid/package.py b/var/spack/repos/builtin/packages/cardioid/package.py index c95035b5bf..6395620d6e 100644 --- a/var/spack/repos/builtin/packages/cardioid/package.py +++ b/var/spack/repos/builtin/packages/cardioid/package.py @@ -25,8 +25,8 @@ class Cardioid(CMakePackage): depends_on('lapack') depends_on('mpi') depends_on('cuda', when="+cuda") - depends_on('mfem+hypre+superlu-dist+lapack', when="+mfem~cuda") - depends_on('mfem+hypre+superlu-dist+lapack^hypre+cuda', when="+mfem+cuda") + depends_on('mfem+mpi+superlu-dist+lapack', when="+mfem~cuda") + depends_on('mfem+mpi+superlu-dist+lapack^hypre+cuda', when="+mfem+cuda") depends_on('cmake@3.1:', type='build') depends_on('perl', type='build') diff --git a/var/spack/repos/builtin/packages/mfem/package.py b/var/spack/repos/builtin/packages/mfem/package.py index 3a4abab49b..1a90ef31b9 100644 --- a/var/spack/repos/builtin/packages/mfem/package.py +++ b/var/spack/repos/builtin/packages/mfem/package.py @@ -92,12 +92,6 @@ class Mfem(Package): # Can we make the default value for 'metis' to depend on the 'mpi' value? variant('metis', default=True, description='Enable METIS support') - # TODO: The 'hypre' variant is the same as 'mpi', we may want to remove it. - # For now, keep the 'hypre' variant while ignoring its setting. This - # is done to preserve compatibility with other packages that refer to - # it, e.g. xSDK. - variant('hypre', default=True, - description='Required for MPI parallelism') variant('openmp', default=False, description='Enable OpenMP parallelism') variant('cuda', default=False, description='Enable CUDA support') -- cgit v1.2.3-70-g09d2