Added Shim to Support hdf5+mpi in Spack Silo (#8130)

* Added Shim to Support hdf5+mpi in Spack Silo

Added compatibility for hdf5+mpi to silo spackage to support spackages
with dependency chains that include silo and hf5+mpi

* Fixed fixable flake8 issue with silo spackage

Fixed missing whitespace around operator

* Setting MPI Variant to Default to True

As per guidance changing variant to default to MPI builds
This commit is contained in:
Robert Pavel 2018-05-14 19:29:03 -06:00 committed by Adam J. Stewart
parent 538d617d14
commit b956608631

View File

@ -42,8 +42,11 @@ class Silo(Package):
description='Builds Silex, a GUI for viewing Silo files')
variant('pic', default=True,
description='Produce position-independent code (for shared libs)')
variant('mpi', default=True,
description='Compile with MPI Compatibility')
depends_on('hdf5~mpi')
depends_on('hdf5~mpi', when='~mpi')
depends_on('hdf5+mpi', when='+mpi')
depends_on('qt', when='+silex')
patch('remove-mpiposix.patch', when='@4.8:4.10.2')
@ -69,6 +72,11 @@ def install(self, spec, prefix):
'CXXFLAGS={0}'.format(self.compiler.pic_flag),
'FCFLAGS={0}'.format(self.compiler.pic_flag)]
if '+mpi' in spec:
config_args.append('CC=%s' % spec['mpi'].mpicc)
config_args.append('CXX=%s' % spec['mpi'].mpicxx)
config_args.append('FC=%s' % spec['mpi'].mpifc)
configure(
'--prefix=%s' % prefix,
'--with-hdf5=%s,%s' % (spec['hdf5'].prefix.include,