From cde5fdde20970d3cd29c55300bff77e68faec120 Mon Sep 17 00:00:00 2001 From: Massimiliano Culpo Date: Fri, 24 Nov 2017 07:15:01 +0100 Subject: Openblas needs FC, not F77 (#6435) fixes #5591 --- var/spack/repos/builtin/packages/openblas/package.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/var/spack/repos/builtin/packages/openblas/package.py b/var/spack/repos/builtin/packages/openblas/package.py index 4019b09935..7c5513f482 100644 --- a/var/spack/repos/builtin/packages/openblas/package.py +++ b/var/spack/repos/builtin/packages/openblas/package.py @@ -98,10 +98,11 @@ class Openblas(MakefilePackage): # 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: + if self.compiler.fc is None: raise InstallError( 'OpenBLAS requires both C and Fortran compilers!' ) + # Add support for OpenMP if (self.spec.satisfies('threads=openmp') and self.spec.satisfies('%clang')): @@ -116,7 +117,7 @@ class Openblas(MakefilePackage): @property def make_defs(self): - # Configure fails to pick up fortran from FC=/abs/path/to/f77, but + # Configure fails to pick up fortran from FC=/abs/path/to/fc, but # works fine with FC=/abs/path/to/gfortran. # When mixing compilers make sure that # $SPACK_ROOT/lib/spack/env/ have symlinks with reasonable @@ -124,7 +125,7 @@ class Openblas(MakefilePackage): make_defs = [ 'CC={0}'.format(spack_cc), - 'FC={0}'.format(spack_f77), + 'FC={0}'.format(spack_fc), 'MAKE_NO_J=1' ] -- cgit v1.2.3-70-g09d2