New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
get rid of hardcoding BLAS libs in easyblocks #155
get rid of hardcoding BLAS libs in easyblocks #155
Conversation
…in.get_variable where appropriate
…CK_MT').split(' -l') for now
…las_opt]/[lapack_opt]
lapack_libs.try_function_by_elem('change', prefix='', prefix_begin_end='-Wl:', | ||
separator=',', separator_begin_end=',') | ||
lapack_libs.SEPARATOR = ',' | ||
lapack = str(lapack_libs) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@boegel isn't this also needed for the fftw configuration? (i saw the easybuildlog of earlier builds that it isn't, dunno if it matters or not)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, good point.
I think it is needed, and that we've been building numpy without FFT support with ictce up until now... I'll look into it, good catch.
@stdweird: Fixed remark, please rerereview. |
might just work, regtests will tell ;) |
Fully tested, Python+numpy_scipy builds as expected with goolf and ictce using this. Merging it in. |
get rid of hardcoding BLAS libs in easyblocks
No description provided.