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
Easyblocks for R extenison patching (including extract) and Extesnion Rm... #435
Easyblocks for R extenison patching (including extract) and Extesnion Rm... #435
Conversation
… Rmpi Intel MPI support (if version >= 0.6-4)
Automatic reply from Jenkins: Can I test this? |
ok to test |
if self.patches: | ||
cmd = "R CMD INSTALL %s %s %s %s --no-clean-on-error" % (self.ext_dir, confargs, confvars, prefix) | ||
else: | ||
cmd = "R CMD INSTALL %s %s %s %s --no-clean-on-error" % (self.ext_src, confargs, confvars, prefix) |
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.
the cmd
value is exactly the same in both cases?!
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.
Look at more close!
cmd = "R CMD INSTALL %s %s %s %s --no-clean-on-error" % (self.ext_dir, confargs, confvars, prefix)
and
cmd = "R CMD INSTALL %s %s %s %s --no-clean-on-error" % (self.ext_src, confargs, confvars, prefix)
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.
Maybe split up the command and the tuple then? The cmd string itself is the same?
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.
Yeah, exactly. Suggestion:
if self.patches:
loc = self.ext_dir
else:
loc = self.ext_src
cmd = "R CMD INSTALL %s %s %s %s --no-clean-on-error" % (loc, confargs, confvars, prefix)
Updated with comments. |
@@ -46,12 +47,19 @@ def run(self): | |||
toolchain.MPI_TYPE_MPICH: "MPICH", | |||
#toolchain.MPI_TYPE_LAM: "LAM", # no support for LAM yet | |||
} | |||
# type of MPI | |||
# MPI_TYPE does not distinguish between MPICH and MPI, which is why we also check mpi_family() |
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.
should say "MPICH and Intel MPI"
Typos and style corrected |
Author name added
Author added
Tested with easybuilders/easybuild-easyconfigs#1077, working great, so going in. Thanks @hajgato! |
…ntel-2014b Easyblocks for R extenison patching (including extract) and Extesnion Rm...
...pi Intel MPI support (if version >= 0.6-4)