Skip to content
This repository has been archived by the owner on Jan 30, 2023. It is now read-only.

Commit

Permalink
Fix a bit of slight sloppiness in variable usage.
Browse files Browse the repository at this point in the history
The "'atlas' in libs" line should really be "'atlas' in libraries"
This was still working due to Python's loop variable leakage, but
from the logic it's clear that the `libraries` variable was intended
to be used here.
  • Loading branch information
embray committed Jul 8, 2016
1 parent db5b361 commit 5384da6
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion build/pkgs/atlas/spkg-install
Expand Up @@ -143,6 +143,7 @@ if 'SAGE_ATLAS_LIB' in os.environ:

paths = [ ATLAS_LIB, os.path.join(ATLAS_LIB, 'lib64'), os.path.join(ATLAS_LIB, 'lib') ]
ATLAS_LIB = None
libraries = []
for libs in libraries_sets:
for path in paths:
if is_atlas_lib_path(path, libs):
Expand Down Expand Up @@ -209,7 +210,7 @@ if 'SAGE_ATLAS_LIB' in os.environ:
for lib in libraries + libraries_optional:
symlinkOSlibrary(prefix+lib)

if 'atlas' in libs:
if 'atlas' in libraries:
write_pc_file(['cblas', 'atlas'], 'cblas')
write_pc_file(['f77blas', 'atlas'], 'blas')
# The inclusion of cblas is not a mistake. ATLAS' lapack include
Expand Down

0 comments on commit 5384da6

Please sign in to comment.