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

Openblas clang fix (#5065)

Browse Source 
* Added a package for the MDAnalysis toolkit.

* Openblas now builds with clang when using 0.2.20

* Fixed flake8 error

* Added a guard to fail for openblas+openmp when on OSX systems

* Changed the guard to look for Apple's build of clang and to improve
the error messages.

* Removed blank line.
This commit is contained in:
Brian Van Essen 2017-08-15 14:57:11 -07:00 committed by Adam J. Stewart
parent 8df9f9e1f4
commit 7b51791625
1 changed files with 9 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
var/spack/repos/builtin/packages/openblas/package.py
View File

@ -84,12 +84,15 @@ def check_compilers(self):
'OpenBLAS requires both C and Fortran compilers!' 'OpenBLAS requires both C and Fortran compilers!'
) )
# Add support for OpenMP # Add support for OpenMP
if '+openmp' in self.spec and self.spec.satisfies('%clang'): if (('+openmp' in self.spec) and self.spec.satisfies('%clang')):
# Openblas (as of 0.2.18) hardcoded that OpenMP cannot if str(self.spec.compiler.version).endswith('-apple'):
# be used with any (!) compiler named clang, bummer. raise InstallError("Apple's clang does not support OpenMP")
raise InstallError( if '@:0.2.19' in self.spec:
'OpenBLAS does not support OpenMP with clang!' # Openblas (as of 0.2.19) hardcoded that OpenMP cannot
) # be used with any (!) compiler named clang, bummer.
raise InstallError(
'OpenBLAS @:0.2.19 does not support OpenMP with clang!'
)
@property @property
def make_defs(self): def make_defs(self):