diff options
Diffstat (limited to 'var/spack/repos/builtin/packages/netcdf-cxx4/package.py')
| -rw-r--r-- | var/spack/repos/builtin/packages/netcdf-cxx4/package.py | 85 |
1 files changed, 42 insertions, 43 deletions
diff --git a/var/spack/repos/builtin/packages/netcdf-cxx4/package.py b/var/spack/repos/builtin/packages/netcdf-cxx4/package.py index cdfe7c4547..9cac19aa7b 100644 --- a/var/spack/repos/builtin/packages/netcdf-cxx4/package.py +++ b/var/spack/repos/builtin/packages/netcdf-cxx4/package.py @@ -14,31 +14,31 @@ class NetcdfCxx4(AutotoolsPackage): sharing of array-oriented scientific data. This is the C++ distribution.""" homepage = "https://www.unidata.ucar.edu/software/netcdf" - url = "ftp://ftp.unidata.ucar.edu/pub/netcdf/netcdf-cxx4-4.3.1.tar.gz" + url = "ftp://ftp.unidata.ucar.edu/pub/netcdf/netcdf-cxx4-4.3.1.tar.gz" - maintainers = ['WardF'] + maintainers = ["WardF"] - version('4.3.1', sha256='6a1189a181eed043b5859e15d5c080c30d0e107406fbb212c8fb9814e90f3445') - version('4.3.0', sha256='e34fbc6aba243ec82c23e9ee99db2430555ada849c54c1f3ab081b0ddd0f5f30') + version("4.3.1", sha256="6a1189a181eed043b5859e15d5c080c30d0e107406fbb212c8fb9814e90f3445") + version("4.3.0", sha256="e34fbc6aba243ec82c23e9ee99db2430555ada849c54c1f3ab081b0ddd0f5f30") - variant('shared', default=True, description='Enable shared library') - variant('pic', default=True, description='Produce position-independent code (for shared libs)') - variant('doc', default=False, description='Enable doxygen docs') + variant("shared", default=True, description="Enable shared library") + variant("pic", default=True, description="Produce position-independent code (for shared libs)") + variant("doc", default=False, description="Enable doxygen docs") - depends_on('netcdf-c') + depends_on("netcdf-c") - depends_on('doxygen', when='+doc', type='build') + depends_on("doxygen", when="+doc", type="build") def flag_handler(self, name, flags): - if name == 'cflags' and '+pic' in self.spec: + if name == "cflags" and "+pic" in self.spec: flags.append(self.compiler.cc_pic_flag) - if name == 'cxxflags' and '+pic' in self.spec: + if name == "cxxflags" and "+pic" in self.spec: flags.append(self.compiler.cxx_pic_flag) - elif name == 'ldlibs': + elif name == "ldlibs": # Address the underlinking problem reported in # https://github.com/Unidata/netcdf-cxx4/issues/86, which also # results into a linking error on macOS: - flags.append(self.spec['netcdf-c'].libs.link_flags) + flags.append(self.spec["netcdf-c"].libs.link_flags) # Note that cflags and cxxflags should be added by the compiler wrapper # and not on the command line to avoid overriding the default @@ -47,31 +47,28 @@ class NetcdfCxx4(AutotoolsPackage): @property def libs(self): - libraries = ['libnetcdf_c++4'] + libraries = ["libnetcdf_c++4"] query_parameters = self.spec.last_query.extra_parameters - if 'shared' in query_parameters: + if "shared" in query_parameters: shared = True - elif 'static' in query_parameters: + elif "static" in query_parameters: shared = False else: - shared = '+shared' in self.spec + shared = "+shared" in self.spec - libs = find_libraries( - libraries, root=self.prefix, shared=shared, recursive=True - ) + libs = find_libraries(libraries, root=self.prefix, shared=shared, recursive=True) if libs: return libs - msg = 'Unable to recursively locate {0} {1} libraries in {2}' + msg = "Unable to recursively locate {0} {1} libraries in {2}" raise spack.error.NoLibrariesError( - msg.format('shared' if shared else 'static', - self.spec.name, - self.spec.prefix)) + msg.format("shared" if shared else "static", self.spec.name, self.spec.prefix) + ) - @when('@4.3.1:+shared') + @when("@4.3.1:+shared") @on_package_attributes(run_tests=True) def patch(self): # We enable the filter tests only when the tests are requested by the @@ -84,22 +81,24 @@ class NetcdfCxx4(AutotoolsPackage): # installations consistent and independent of whether the shared # libraries or the tests are requested, we prevent installation of # 'libh5bzip2.so': - filter_file(r'(^\s*)lib(_LTLIBRARIES\s*)(=\s*libh5bzip2\.la\s*$)', - r'\1noinst\2+\3', join_path(self.stage.source_path, - 'plugins', 'Makefile.in')) + filter_file( + r"(^\s*)lib(_LTLIBRARIES\s*)(=\s*libh5bzip2\.la\s*$)", + r"\1noinst\2+\3", + join_path(self.stage.source_path, "plugins", "Makefile.in"), + ) def configure_args(self): - config_args = self.enable_or_disable('shared') + config_args = self.enable_or_disable("shared") - if '+doc' in self.spec: - config_args.append('--enable-doxygen') + if "+doc" in self.spec: + config_args.append("--enable-doxygen") else: - config_args.append('--disable-doxygen') + config_args.append("--disable-doxygen") - if self.spec.satisfies('@4.3.1:'): - if self.run_tests and '+shared' in self.spec: - config_args.append('--enable-filter-testing') - if self.spec.satisfies('^hdf5+mpi'): + if self.spec.satisfies("@4.3.1:"): + if self.run_tests and "+shared" in self.spec: + config_args.append("--enable-filter-testing") + if self.spec.satisfies("^hdf5+mpi"): # The package itself does not need the MPI libraries but # includes <hdf5.h> in the filter test C code, which # requires <mpi.h> when HDF5 is built with the MPI support. @@ -110,13 +109,13 @@ class NetcdfCxx4(AutotoolsPackage): # MPICH might reside in /opt/local/include/mpich-gcc10, # which Spack does not know about and cannot inject with its # compiler wrapper. - config_args.append('CC={0}'.format(self.spec['mpi'].mpicc)) + config_args.append("CC={0}".format(self.spec["mpi"].mpicc)) else: - config_args.append('--disable-filter-testing') + config_args.append("--disable-filter-testing") return config_args - @run_after('configure') + @run_after("configure") def rename_version(self): # See https://github.com/Unidata/netcdf-cxx4/issues/109 # The issue is fixed upstream: @@ -129,14 +128,14 @@ class NetcdfCxx4(AutotoolsPackage): # patch method because writing a robust regexp seems to be more # difficult that simply renaming the file if exists. It also looks like # we can simply remove the file since it is not used anywhere. - if not self.spec.satisfies('@:4.3.1 platform=darwin'): + if not self.spec.satisfies("@:4.3.1 platform=darwin"): return with working_dir(self.build_directory): - fname = 'VERSION' + fname = "VERSION" if os.path.exists(fname): - os.rename(fname, '{0}.txt'.format(fname)) + os.rename(fname, "{0}.txt".format(fname)) def check(self): with working_dir(self.build_directory): - make('check', parallel=False) + make("check", parallel=False) |