zhangyiss/spack
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

BLD: enforce C++11 std for boost + xl_r (#13829)

Browse Source 
* BLD: enforce C++11 std for boost + xl_r

* the spack `cxxstd` variant is not sufficient to enforce
`-std=c++11` usage in boost compile lines when `xl_r` compiler
spec is in use; while it would be nice if this were fixed
in a boost config file somewhere, for now this patch
allows boost to build on POWER9 with
an %xl_r compiler spec if the user specifies i.e.,:

`spack install boost@1.70.0+mpi cxxstd=11 %xl_r@16.1.1.5`

* Update var/spack/repos/builtin/packages/boost/package.py

Co-Authored-By: Adam J. Stewart <ajstewart426@gmail.com>
This commit is contained in:
Tyler Reddy 2019-11-27 22:08:54 -07:00 committed by Adam J. Stewart
parent e974f48be0
commit 008fe37941
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
var/spack/repos/builtin/packages/boost/package.py
View File

@ -367,6 +367,12 @@ def determine_b2_options(self, spec, options):
cxxflags.append('-stdlib=libc++')
options.extend(['toolset=clang',
'linkflags="-stdlib=libc++"'])
elif spec.satisfies('%xl') or spec.satisfies('%xl_r'):
# see also: https://lists.boost.org/boost-users/2019/09/89953.php
# the cxxstd setting via spack is not sufficient to drive the
# change into boost compilation
if spec.variants['cxxstd'].value == '11':
cxxflags.append('-std=c++11')
if cxxflags:
options.append('cxxflags="{0}"'.format(' '.join(cxxflags)))