Fixed the "+shared" variant in the "hdf5" package.
This commit is contained in:
parent
0388093f7a
commit
6cd76d69c3
2 changed files with 5 additions and 4 deletions
|
@ -23,7 +23,7 @@ class Exodusii(Package):
|
|||
# TODO: Make this a build dependency once build dependencies are supported
|
||||
# (see: https://github.com/LLNL/spack/pull/378).
|
||||
depends_on('cmake@2.8.7:')
|
||||
depends_on('hdf5+shared~mpi')
|
||||
depends_on('hdf5~shared~mpi')
|
||||
depends_on('netcdf~mpi')
|
||||
|
||||
patch('exodus-cmake.patch')
|
||||
|
|
|
@ -42,7 +42,7 @@ class Hdf5(Package):
|
|||
version('1.8.13', 'c03426e9e77d7766944654280b467289')
|
||||
|
||||
variant('debug', default=False, description='Builds a debug version of the library')
|
||||
variant('shared', default=False, description='Builds a static executable version of the library')
|
||||
variant('shared', default=True, description='Builds a shared version of the library')
|
||||
|
||||
variant('cxx', default=True, description='Enable C++ support')
|
||||
variant('fortran', default=True, description='Enable Fortran support')
|
||||
|
@ -79,7 +79,9 @@ def install(self, spec, prefix):
|
|||
else:
|
||||
extra_args.append('--enable-production')
|
||||
|
||||
if '+static' in spec:
|
||||
if '+shared' in spec:
|
||||
extra_args.append('--enable-shared')
|
||||
else:
|
||||
extra_args.append('--enable-static-exec')
|
||||
|
||||
if '+unsupported' in spec:
|
||||
|
@ -123,7 +125,6 @@ def install(self, spec, prefix):
|
|||
configure(
|
||||
"--prefix=%s" % prefix,
|
||||
"--with-zlib=%s" % spec['zlib'].prefix,
|
||||
"--enable-shared", # TODO : this should be enabled by default, remove it?
|
||||
*extra_args)
|
||||
make()
|
||||
make("install")
|
||||
|
|
Loading…
Reference in a new issue