diff options
author | Seth R. Johnson <johnsonsr@ornl.gov> | 2021-05-19 06:59:06 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-05-19 06:59:06 -0400 |
commit | 3d2cd480486648cd60c2400b9771df34e7aca684 (patch) | |
tree | c4cda21621fba37732020be3a10eb1b0df84c9eb /var/spack/repos/builtin/packages/aspect/package.py | |
parent | 33df3990de3dc89fcdbefd1cb4b8bfe0b316f43a (diff) | |
download | spack-3d2cd480486648cd60c2400b9771df34e7aca684.tar.gz spack-3d2cd480486648cd60c2400b9771df34e7aca684.tar.bz2 spack-3d2cd480486648cd60c2400b9771df34e7aca684.tar.xz spack-3d2cd480486648cd60c2400b9771df34e7aca684.zip |
Use define_from_variant in numerous CMakePackages (#23655)
Example replacement:
```
'-D(\w+)(:BOOL)?=\{0\}'\.\s*format\s*\(\s*'(ON|YES|true|TRUE)' if '\+(\w+)' in (self\.)?spec else '(OFF|NO|false|FALSE)'\)
```
with
```
self.define_from_variant('\1', '\4')
```
This will cause failures if any variants were misspelled: I have already caught two packages with nonexistent variants.
Diffstat (limited to 'var/spack/repos/builtin/packages/aspect/package.py')
-rw-r--r-- | var/spack/repos/builtin/packages/aspect/package.py | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/var/spack/repos/builtin/packages/aspect/package.py b/var/spack/repos/builtin/packages/aspect/package.py index f996313c98..efb17a0be6 100644 --- a/var/spack/repos/builtin/packages/aspect/package.py +++ b/var/spack/repos/builtin/packages/aspect/package.py @@ -35,8 +35,7 @@ class Aspect(CMakePackage): def cmake_args(self): return [ - '-DASPECT_USE_FP_EXCEPTIONS=%s' % - ('ON' if '+fpe' in self.spec else 'OFF') + self.define_from_variant('ASPECT_USE_FP_EXCEPTIONS', 'fpe') ] def setup_run_environment(self, env): |