-
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
added easyblock for oasyslsdyna #1363
Conversation
super(EB_OasysLSDyna, self).__init__(*args, **kwargs) | ||
|
||
def install_step(self): | ||
cmd = "echo -e \"%s\" | ./setup.csh" % ('\\n'.join(['FULL', 'Y', self.installdir, 'Y', 'N', 'Y', self.installdir, 'Y'])) |
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.
@mboisson Please use run_cmd_qa
for this
import shutil | ||
import sys | ||
import tempfile | ||
from distutils.version import LooseVersion |
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.
@mboisson Please remove any imports you're not using, e.g. fileinput
, LooseVersion
, etc. Same below
|
||
def __init__(self,*args,**kwargs): | ||
"""Enable building in install dir.""" | ||
super(EB_OasysLSDyna, self).__init__(*args, **kwargs) |
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.
If you're not doing anything custom, you can drop the custom constructor.
def make_module_extra(self): | ||
"""Correctly prepend PATH.""" | ||
|
||
txt = super(EB_OasysLSDyna, self).make_module_extra() |
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.
Same here, not customization being done, so this method can be dropped
@mboisson Any progress on this? |
No, I have not had time to come back to this, especially changing to use run_qmd_qa. The other changes are trivial, but this one requires rerunning the recipe and fiddling with the right parameters to use. |
Work in progress, but I would like comments anyway.