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

Merge pull request #1079 from davydden/pkg/blas_mpi_fortran

Browse Source 
require fortran compilers for Openblas and Openmpi
This commit is contained in:
Todd Gamblin
2016-06-20 09:36:23 -07:00
committed by GitHub
parent ce105e9bbe cf4a34c657
commit 8bdb6695c7
2 changed files with 14 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -48,6 +48,13 @@ class Openblas(Package):
patch('make.patch') patch('make.patch')
def install(self, spec, prefix): def install(self, spec, prefix):
# As of 06/2016 there is no mechanism to specify that packages which
# depends on Blas/Lapack need C or/and Fortran symbols. For now
# require both.
if self.compiler.f77 is None:
raise InstallError('OpenBLAS requires both C and Fortran ',
'compilers!')
# Configure fails to pick up fortran from FC=/abs/path/to/f77, but # Configure fails to pick up fortran from FC=/abs/path/to/f77, but
# works fine with FC=/abs/path/to/gfortran. # works fine with FC=/abs/path/to/gfortran.
# When mixing compilers make sure that # When mixing compilers make sure that

7
var/spack/repos/builtin/packages/openmpi/package.py
View File

@@ -121,6 +121,13 @@ def verbs(self):
return 'verbs' return 'verbs'
def install(self, spec, prefix): def install(self, spec, prefix):
# As of 06/2016 there is no mechanism to specify that packages which
# depends on MPI need C or/and Fortran implementation. For now
# require both.
if (self.compiler.f77 is None) or (self.compiler.fc is None):
raise InstallError('OpenMPI requires both C and Fortran ',
'compilers!')
config_args = ["--prefix=%s" % prefix, config_args = ["--prefix=%s" % prefix,
"--with-hwloc=%s" % spec['hwloc'].prefix, "--with-hwloc=%s" % spec['hwloc'].prefix,
"--enable-shared", "--enable-shared",