From e4edcf610436ba6d87865763bbc442952ac7d889 Mon Sep 17 00:00:00 2001 From: Arnur Nigmetov Date: Fri, 24 Mar 2023 12:28:20 -0700 Subject: henson: simplify args logic with define_from_variant (#36398) Co-authored-by: Arnur Nigmetov --- var/spack/repos/builtin/packages/henson/package.py | 17 +++++++---------- 1 file changed, 7 insertions(+), 10 deletions(-) (limited to 'var') 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 -- cgit v1.2.3-60-g2f50