diff options
| author | Mark W. Krentel <krentel@rice.edu> | 2019-12-09 17:11:47 -0600 |
|---|---|---|
| committer | Adam J. Stewart <ajstewart426@gmail.com> | 2019-12-09 17:11:47 -0600 |
| commit | c627e5a6fcefb44330b4980835f0cf3a34ff2944 (patch) | |
| tree | 9bd306363b54a9741cb3b7c1a197bfaaa051aa9b /var | |
| parent | 0592c580302ca8edf6f404d046d2e670a69bba6e (diff) | |
| download | spack-c627e5a6fcefb44330b4980835f0cf3a34ff2944.tar.gz spack-c627e5a6fcefb44330b4980835f0cf3a34ff2944.tar.bz2 spack-c627e5a6fcefb44330b4980835f0cf3a34ff2944.tar.xz spack-c627e5a6fcefb44330b4980835f0cf3a34ff2944.zip | |
intel-tbb: fix and cleanup patch version ranges (#14053)
Recent commit e9ee9eaf (#13989) fixed testing version ranges inside
patch when clauses. Previously, it was necessary to write all revs
individually for packages with multiple length version numbers (2019
and 2019.1).
This fixes the build for the old 2017.* versions.
Diffstat (limited to 'var')
| -rw-r--r-- | var/spack/repos/builtin/packages/intel-tbb/package.py | 27 |
1 files changed, 11 insertions, 16 deletions
diff --git a/var/spack/repos/builtin/packages/intel-tbb/package.py b/var/spack/repos/builtin/packages/intel-tbb/package.py index 115f980e69..5db08142d9 100644 --- a/var/spack/repos/builtin/packages/intel-tbb/package.py +++ b/var/spack/repos/builtin/packages/intel-tbb/package.py @@ -81,26 +81,21 @@ class IntelTbb(Package): variant('tm', default=True, description='Enable use of transactional memory on x86') - # Build and install CMake config files if we're new enough. - depends_on('cmake@3.0.0:', type='build', when='@2017.0:') + # Testing version ranges inside when clauses was fixed in e9ee9eaf. + # See: #8957 and #13989. - # Note: see issues #11371 and #8957 to understand why 2019.x patches are - # specified one at a time. In a nutshell, it is currently impossible - # to patch `2019.1` without patching `2019`. When #8957 is fixed, this - # can be simplified. + # Build and install CMake config files if we're new enough. + # CMake support started in 2017.7. + depends_on('cmake@3.0.0:', type='build', when='@2017.7:') # Patch for pedantic warnings (#10836). This was fixed in the TBB - # source tree in 2019.6. One patch file for 2019.1 and after... - patch("gcc_generic-pedantic-2019.patch", level=1, - when='@2019.1,2019.2,2019.3,2019.4,2019.5') - # ...another patch file for 2019 and before - patch("gcc_generic-pedantic-4.4.patch", level=1, when='@:2019.0') + # source tree in 2019.6. + patch("gcc_generic-pedantic-2019.patch", level=1, when='@2019.1:2019.5') + patch("gcc_generic-pedantic-4.4.patch", level=1, when='@:2019.0') # Patch cmakeConfig.cmake.in to find the libraries where we install them. - # Can't use '@2019.5:' because 2019.5 <= 2019 <= 2019.99. wtf!? - patch("tbb_cmakeConfig-2019.5.patch", level=0, - when='@2019.5,2019.6,2019.7,2019.8') - patch("tbb_cmakeConfig.patch", level=0, when='@2017.0:2019.4') + patch("tbb_cmakeConfig-2019.5.patch", level=0, when='@2019.5:') + patch("tbb_cmakeConfig.patch", level=0, when='@2017.7:2019.4') # Some very old systems don't support transactional memory. patch("disable-tm.patch", when='~tm') @@ -197,7 +192,7 @@ class IntelTbb(Package): for f in fs: install(f, prefix.lib) - if self.spec.satisfies('@2017.0:'): + if spec.satisfies('@2017.8,2018.1:', strict=True): # Generate and install the CMake Config file. cmake_args = ('-DTBB_ROOT={0}'.format(prefix), '-DTBB_OS={0}'.format(platform.system()), |