From 526ae841371fcf344bb0c35e881198bc4e86641c Mon Sep 17 00:00:00 2001 From: Axel Huebl Date: Tue, 27 Jun 2023 22:42:57 +0200 Subject: HDF5: is_enabled helper (ON) (#35705) * HDF5: is_enabled helper (ON) Slightly generalize the `is_enabled` helper in the HDF5 package. `ON` is the most typical CMake bool option passed, besides many other possible `true` values, and should be included as a possible check to the config. * Simplify --------- Co-authored-by: Tamara Dahlgren <35777542+tldahlgren@users.noreply.github.com> --- var/spack/repos/builtin/packages/hdf5/package.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/var/spack/repos/builtin/packages/hdf5/package.py b/var/spack/repos/builtin/packages/hdf5/package.py index 1a48d47158..dcff5904eb 100644 --- a/var/spack/repos/builtin/packages/hdf5/package.py +++ b/var/spack/repos/builtin/packages/hdf5/package.py @@ -489,9 +489,7 @@ class Hdf5(CMakePackage): @classmethod def determine_variants(cls, exes, version): def is_enabled(text): - if text in set(["t", "true", "enabled", "yes", "1"]): - return True - return False + return text.lower() in ["t", "true", "enabled", "yes", "1", "on"] results = [] for exe in exes: -- cgit v1.2.3-70-g09d2