diff options
| author | Arnur Nigmetov <a.nigmetov@gmail.com> | 2023-03-24 12:28:20 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-03-24 15:28:20 -0400 |
| commit | e4edcf610436ba6d87865763bbc442952ac7d889 (patch) | |
| tree | 81894bc1193504c6f96425bfceb81ef93372ff29 /var | |
| parent | 693eea499c707b73a2f9d5f389963885876cc1de (diff) | |
| download | spack-e4edcf610436ba6d87865763bbc442952ac7d889.tar.gz spack-e4edcf610436ba6d87865763bbc442952ac7d889.tar.bz2 spack-e4edcf610436ba6d87865763bbc442952ac7d889.tar.xz spack-e4edcf610436ba6d87865763bbc442952ac7d889.zip | |
henson: simplify args logic with define_from_variant (#36398)
Co-authored-by: Arnur Nigmetov <nigmetov@tugraz.at>
Diffstat (limited to 'var')
| -rw-r--r-- | var/spack/repos/builtin/packages/henson/package.py | 17 |
1 files changed, 7 insertions, 10 deletions
diff --git a/var/spack/repos/builtin/packages/henson/package.py b/var/spack/repos/builtin/packages/henson/package.py index 4f66c00732..34dc0a63bb 100644 --- a/var/spack/repos/builtin/packages/henson/package.py +++ b/var/spack/repos/builtin/packages/henson/package.py @@ -26,15 +26,12 @@ class Henson(CMakePackage): conflicts("^openmpi", when="+mpi-wrappers") def cmake_args(self): - args = [] - if "+python" in self.spec: - args += ["-Dpython=on"] - else: - args += ["-Dpython=off"] - - if "+mpi-wrappers" in self.spec: - args += ["-Dmpi-wrappers=on"] - else: - args += ["-Dmpi-wrappers=off"] + args = [ + self.define_from_variant("python", "python"), + self.define_from_variant("mpi-wrappers", "mpi-wrappers"), + ] + + if self.spec.satisfies("+python"): + args += [self.define("PYTHON_EXECUTABLE", self.spec["python"].command.path)] return args |