-
Notifications
You must be signed in to change notification settings - Fork 284
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 easyblocks for GROMACS + minor bug fix in IntelBase easyblock #335
Conversation
… building with OpenMP support enabled, specify home of CUDA explicitely, only checking for BLAS/LAPACK/FFT for recent GROMACS versions
@@ -148,7 +148,8 @@ def setup_local_home_subdir(self): | |||
self.home_subdir_local) | |||
if not symlink_ok: | |||
# rename current 'intel' dir | |||
home_intel_bk = '.'.join([self.home_subdir, "bk_easybuild"]) | |||
home_intel_bk = tempfile.mkdtemp(dir=os.path.dirname(self.home_subdir), | |||
prefix='%s.bk.' % os.path.basename(self.home_subdir)) |
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.
why is this update in this PR? not clear...
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.
it's hitch hiking here because it's a bug fix, and I expected this to go in fast (yeah, I know)
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.
ok, that explains it; since the PR affects the newcoming ictce GROMACS builds, I think it's passable as such...
if tests pass on your side, let's call this OK for merging... |
This reverts commit 0715fcc.
well-tested with the easyconfigs touched in easybuilders/easybuild-easyconfigs#606, good to go thanks for the review @fgeorgatos! |
add easyblocks for GROMACS + minor bug fix in IntelBase easyblock
No description provided.