summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGreg Lee <lee218@llnl.gov>2016-07-05 14:28:24 -0700
committerGreg Lee <lee218@llnl.gov>2016-07-06 11:51:32 -0700
commit4e27d91351f1f2f29adec9d1be44e25cbb79bb9f (patch)
tree6ed1f24dc76a425e4ba0518e205c8db28ff6dc3c
parent4c063c60556d714fd7fe4792b6aa137fd7d3f45e (diff)
downloadspack-4e27d91351f1f2f29adec9d1be44e25cbb79bb9f.tar.gz
spack-4e27d91351f1f2f29adec9d1be44e25cbb79bb9f.tar.bz2
spack-4e27d91351f1f2f29adec9d1be44e25cbb79bb9f.tar.xz
spack-4e27d91351f1f2f29adec9d1be44e25cbb79bb9f.zip
flake 8 fixes
-rw-r--r--var/spack/repos/builtin/packages/suite-sparse/package.py31
1 files changed, 16 insertions, 15 deletions
diff --git a/var/spack/repos/builtin/packages/suite-sparse/package.py b/var/spack/repos/builtin/packages/suite-sparse/package.py
index bc25fa8c6c..1dc64f385f 100644
--- a/var/spack/repos/builtin/packages/suite-sparse/package.py
+++ b/var/spack/repos/builtin/packages/suite-sparse/package.py
@@ -44,32 +44,33 @@ class SuiteSparse(Package):
depends_on('metis@5.1.0', when='@4.5.1')
depends_on('metis@5.1.0', when='@4.5.3')
# FIXME:
- # in @4.5.1. TBB support in SPQR seems to be broken as TBB-related linkng flags
- # does not seem to be used, which leads to linking errors on Linux.
+ # in @4.5.1. TBB support in SPQR seems to be broken as TBB-related linkng
+ # flags does not seem to be used, which leads to linking errors on Linux.
# Try re-enabling in future versions.
# depends_on('tbb', when='+tbb')
def install(self, spec, prefix):
- # The build system of SuiteSparse is quite old-fashioned
- # It's basically a plain Makefile which include an header (SuiteSparse_config/SuiteSparse_config.mk)
- # with a lot of convoluted logic in it.
- # Any kind of customization will need to go through filtering of that file
+ # The build system of SuiteSparse is quite old-fashioned.
+ # It's basically a plain Makefile which include an header
+ # (SuiteSparse_config/SuiteSparse_config.mk)with a lot of convoluted
+ # logic in it. Any kind of customization will need to go through
+ # filtering of that file
make_args = ['INSTALL=%s' % prefix]
# inject Spack compiler wrappers
make_args.extend([
- 'AUTOCC=no',
- 'CC=cc',
- 'CXX=c++',
- 'F77=f77',
+ 'AUTOCC=no',
+ 'CC=cc',
+ 'CXX=c++',
+ 'F77=f77',
])
# use Spack's metis in CHOLMOD/Partition module,
# otherwise internal Metis will be compiled
make_args.extend([
- 'MY_METIS_LIB=-L%s -lmetis' % spec['metis'].prefix.lib,
- 'MY_METIS_INC=%s' % spec['metis'].prefix.include,
+ 'MY_METIS_LIB=-L%s -lmetis' % spec['metis'].prefix.lib,
+ 'MY_METIS_INC=%s' % spec['metis'].prefix.include,
])
# Intel TBB in SuiteSparseQR
@@ -80,10 +81,10 @@ class SuiteSparse(Package):
])
# BLAS arguments require path to libraries
- # FIXME : (blas / lapack always provide libblas and liblapack as aliases)
+ # FIXME: (blas/lapack always provide libblas and liblapack as aliases)
if '@4.5.1' in spec:
- # adding -lstdc++ is clearly an ugly way to do this, but it follows suit
- # with the TCOV path of SparseSuite 4.5.1's Suitesparse_config.mk
+ # adding -lstdc++ is clearly an ugly way to do this, but it follows
+ # with the TCOV path of SparseSuite 4.5.1's Suitesparse_config.mk
make_args.extend([
'BLAS=-lblas -lstdc++',
'LAPACK=-llapack'