diff options
author | Sergey Kosukhin <sergey.kosukhin@mpimet.mpg.de> | 2023-07-13 19:53:19 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-07-13 13:53:19 -0400 |
commit | afe6f7ed79faf15c35916f15be07ab5d20d06cb1 (patch) | |
tree | 19d341b491b1aaf9d70b1404bff5ee78af4afa42 /var | |
parent | 82d41a7be478e001d5653d831c1f204fb7971efe (diff) | |
download | spack-afe6f7ed79faf15c35916f15be07ab5d20d06cb1.tar.gz spack-afe6f7ed79faf15c35916f15be07ab5d20d06cb1.tar.bz2 spack-afe6f7ed79faf15c35916f15be07ab5d20d06cb1.tar.xz spack-afe6f7ed79faf15c35916f15be07ab5d20d06cb1.zip |
eccodes: fix a few issues in the recipe (#38873)
* explicitly disable the Python 2 interface
* drop obsolete cmake argument HDF5_ROOT for newer versions
* set PYTHON_EXECUTABLE only when needed
Diffstat (limited to 'var')
-rw-r--r-- | var/spack/repos/builtin/packages/eccodes/package.py | 22 |
1 files changed, 12 insertions, 10 deletions
diff --git a/var/spack/repos/builtin/packages/eccodes/package.py b/var/spack/repos/builtin/packages/eccodes/package.py index 7afadcbe80..05ad38f9b1 100644 --- a/var/spack/repos/builtin/packages/eccodes/package.py +++ b/var/spack/repos/builtin/packages/eccodes/package.py @@ -321,6 +321,9 @@ class Eccodes(CMakePackage): self.define_from_variant("ENABLE_ECCODES_THREADS", "pthreads"), self.define_from_variant("ENABLE_ECCODES_OMP_THREADS", "openmp"), self.define_from_variant("ENABLE_MEMFS", "memfs"), + self.define( + "ENABLE_PYTHON{0}".format("2" if self.spec.satisfies("@2.20.0:") else ""), False + ), self.define_from_variant("ENABLE_FORTRAN", "fortran"), self.define("BUILD_SHARED_LIBS", "BOTH" if "+shared" in self.spec else "OFF"), self.define("ENABLE_TESTS", self.run_tests), @@ -332,15 +335,14 @@ class Eccodes(CMakePackage): ] if "+netcdf" in self.spec: - args.extend( - [ - # Prevent possible overriding by environment variables - # NETCDF_ROOT, NETCDF_DIR, and NETCDF_PATH: - self.define("NETCDF_PATH", self.spec["netcdf-c"].prefix), - # Prevent overriding by environment variable HDF5_ROOT: - self.define("HDF5_ROOT", self.spec["hdf5"].prefix), - ] - ) + # Prevent possible overriding by environment variables NETCDF_ROOT, NETCDF_DIR, and + # NETCDF_PATH: + args.append(self.define("NETCDF_PATH", self.spec["netcdf-c"].prefix)) + # Prevent overriding by environment variable HDF5_ROOT (starting version 2.14.0, + # ecCodes is shipped with ecBuild 3.1.0+, which does not seem to rely on the HDF5_ROOT + # variable): + if self.spec.satisfies("@:2.13"): + args.append(self.define("HDF5_ROOT", self.spec["hdf5"].prefix)) if jp2k == "openjpeg": args.append(self.define("OPENJPEG_PATH", self.spec["openjpeg"].prefix)) @@ -352,7 +354,7 @@ class Eccodes(CMakePackage): # Prevent overriding by environment variables AEC_DIR and AEC_PATH: args.append(self.define("AEC_DIR", self.spec["libaec"].prefix)) - if "^python" in self.spec: + if "+memfs" in self.spec: args.append(self.define("PYTHON_EXECUTABLE", python.path)) definitions = self.spec.variants["definitions"].value |