Skip to content
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

Add mkl 11.3.1.150 support #745

Closed
wants to merge 4 commits into from
Closed
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion easybuild/easyblocks/i/imkl.py
Expand Up @@ -357,7 +357,7 @@ def sanity_check_step(self):
if self.cfg['m32']:
raise EasyBuildError("Sanity check for 32-bit not implemented yet for IMKL v%s (>= 10.3)", self.version)
else:
mkldirs = ["bin", "mkl/bin", "mkl/bin/intel64", "mkl/lib/intel64", "mkl/include"]
mkldirs = ["bin", "mkl/bin", "mkl/lib/intel64", "mkl/include"]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should only remove the check on mkl/bin/intel64 for the newest versions of imkl, and also check what kind of potential fallout this has (which tools are located in mkl/bin/intel, and where are they used by EB)?

libs += [lib % {'suff': suff} for lib in extralibs for suff in ['lp64', 'ilp64']]
mklfiles = ["mkl/lib/intel64/libmkl.so", "mkl/include/mkl.h"] + \
["mkl/lib/intel64/%s" % lib for lib in libs]
Expand Down