hlrs-software-stack/spack
7
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

intel-tbb: fix and cleanup patch version ranges (#14053)

Browse source 
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.
This commit is contained in:
Mark W. Krentel 2019-12-09 17:11:47 -06:00 committed by Adam J. Stewart
parent 0592c58030
commit c627e5a6fc
1 changed files with 11 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

27
var/spack/repos/builtin/packages/intel-tbb/package.py
View file

@ -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 @@ def install(self, spec, prefix):
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()),