zhangyiss/spack
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

mfem does not use cmake for builds [anymore] - so remove code corresponding to this dependency. (#6073)

Browse Source 
This enables mfem build on OSX [with xcode/clang compilers]
This commit is contained in:
Satish Balay 2017-11-01 10:52:03 -05:00 committed by Todd Gamblin
parent b53adc90f7
commit 445ffc46a1
1 changed files with 0 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
var/spack/repos/builtin/packages/mfem/package.py
View File

@ -126,7 +126,6 @@ class Mfem(Package):
depends_on('petsc@3.8:', when='+petsc') depends_on('petsc@3.8:', when='+petsc')
depends_on('mpfr', when='+mpfr') depends_on('mpfr', when='+mpfr')
depends_on('cmake', when='^metis@5:', type='build')
depends_on('netcdf', when='@3.2: +netcdf') depends_on('netcdf', when='@3.2: +netcdf')
depends_on('zlib', when='@3.2: +netcdf') depends_on('zlib', when='@3.2: +netcdf')
depends_on('hdf5', when='@3.2: +netcdf') depends_on('hdf5', when='@3.2: +netcdf')
@ -135,14 +134,6 @@ class Mfem(Package):
patch('mfem_ppc_build.patch', when='@3.2:3.3 arch=ppc64le') patch('mfem_ppc_build.patch', when='@3.2:3.3 arch=ppc64le')
def check_variants(self, spec):
if 'metis@5:' in spec and '%clang' in spec and (
'^cmake %gcc' not in spec):
raise InstallError('To work around CMake bug with clang, must ' +
'build mfem with mfem[+variants] %clang ' +
'^cmake %gcc to force CMake to build with gcc')
return
# #
# Note: Although MFEM does support CMake configuration, MFEM # Note: Although MFEM does support CMake configuration, MFEM
# development team indicates that vanilla GNU Make is the # development team indicates that vanilla GNU Make is the
@ -151,7 +142,6 @@ def check_variants(self, spec):
# configuration options. So, don't use CMake # configuration options. So, don't use CMake
# #
def install(self, spec, prefix): def install(self, spec, prefix):
self.check_variants(spec)
def yes_no(varstr): def yes_no(varstr):
return 'YES' if varstr in self.spec else 'NO' return 'YES' if varstr in self.spec else 'NO'