diff options
| author | Mark W. Krentel <krentel@rice.edu> | 2019-05-09 13:58:41 -0500 |
|---|---|---|
| committer | Peter Scheibel <scheibel1@llnl.gov> | 2019-05-10 03:58:41 +0900 |
| commit | 26b9369c577a3c075f8fe220bbe1582a47936cc8 (patch) | |
| tree | 9e0fa3c9d4444eddf7d44777c1acfda052168e51 | |
| parent | ac3cbfd69ac7c1112962a2f1141bd6fbb4d29520 (diff) | |
| download | spack-26b9369c577a3c075f8fe220bbe1582a47936cc8.tar.gz spack-26b9369c577a3c075f8fe220bbe1582a47936cc8.tar.bz2 spack-26b9369c577a3c075f8fe220bbe1582a47936cc8.tar.xz spack-26b9369c577a3c075f8fe220bbe1582a47936cc8.zip | |
dyninst: update cmake args and boost dependency (#11378)
* Starting with the upcoming 10.1 release, Dyninst changed the names of
some cmake args to camel case: LIBELF_* to LibElf_* and LIBDWARF_* to
LibDwarf_*. This change is needed now for @develop and will be used
for 10.1 when it is released.
* New versions of Dyninst now requires cmake version 3.4.0 or later.
* Restrict boost dependency version to < 1.70.0 to avoid build error
| -rw-r--r-- | var/spack/repos/builtin/packages/dyninst/package.py | 26 |
1 files changed, 18 insertions, 8 deletions
diff --git a/var/spack/repos/builtin/packages/dyninst/package.py b/var/spack/repos/builtin/packages/dyninst/package.py index 8db6ac644f..1e370557f2 100644 --- a/var/spack/repos/builtin/packages/dyninst/package.py +++ b/var/spack/repos/builtin/packages/dyninst/package.py @@ -34,7 +34,7 @@ class Dyninst(CMakePackage): boost_libs = '+atomic+chrono+date_time+filesystem+system+thread+timer' - depends_on('boost@1.61.0:' + boost_libs) + depends_on('boost@1.61.0:1.69.99' + boost_libs) depends_on('libiberty+pic') # Dyninst uses elf@1 (elfutils) starting with 9.3.0, and used @@ -48,7 +48,7 @@ class Dyninst(CMakePackage): depends_on('tbb@2018.6:', when='@10.0:') - depends_on('cmake@3.0:', type='build', when='@10.0:') + depends_on('cmake@3.4.0:', type='build', when='@10.0:') depends_on('cmake@2.8:', type='build', when='@:9.99') patch('stat_dysect.patch', when='+stat_dysect') @@ -84,14 +84,24 @@ class Dyninst(CMakePackage): args = [ '-DPATH_BOOST=%s' % spec['boost'].prefix, '-DIBERTY_LIBRARIES=%s' % spec['libiberty'].libs, - - '-DLIBELF_INCLUDE_DIR=%s' % elf_include, - '-DLIBELF_LIBRARIES=%s' % spec['elf'].libs, - - '-DLIBDWARF_INCLUDE_DIR=%s' % dwarf_include, - '-DLIBDWARF_LIBRARIES=%s' % dwarf_lib, ] + # 10.1 changed the spelling of LibElf and LibDwarf. + if spec.satisfies('@10.1.0:'): + args.extend([ + '-DLibElf_INCLUDE_DIR=%s' % elf_include, + '-DLibElf_LIBRARIES=%s' % spec['elf'].libs, + '-DLibDwarf_INCLUDE_DIR=%s' % dwarf_include, + '-DLibDwarf_LIBRARIES=%s' % dwarf_lib, + ]) + else: + args.extend([ + '-DLIBELF_INCLUDE_DIR=%s' % elf_include, + '-DLIBELF_LIBRARIES=%s' % spec['elf'].libs, + '-DLIBDWARF_INCLUDE_DIR=%s' % dwarf_include, + '-DLIBDWARF_LIBRARIES=%s' % dwarf_lib, + ]) + # TBB include and lib directories, version 10.x or later. if spec.satisfies('@10.0.0:'): args.extend([ |