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
modeller easyblock #392
modeller easyblock #392
Conversation
|
||
from easybuild.framework.easyblock import EasyBlock | ||
from easybuild.tools.filetools import run_cmd_qa | ||
from easybuild.framework.easyconfig import CUSTOM |
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.
put these in alphabetical order
Looks good |
def extra_options(): | ||
"""Add extra easyconfig parameters custom to Modeller""" | ||
extra_vars = { | ||
'LICENSEKEY': ["", "Specify the modeller license key", CUSTOM], |
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, I would rather like to see this as a generic easyconfig parameter license_key
, next to the existing license_file
and license_server
, since it's a generic concept
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.
I don't know exactly what you mean...
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.
we should add license_key
as an available easyconfig parameter in framework, rather than doing it only here in this easyblock...
@pescobar: ping on these remarks? |
@pescobar: ping wrt resolving remarks? |
@boegel I will take a look at this on monday |
add copyright/license header, use license_key easyconfig parameter
use existing 'key' easyconfig parameter
tested with easybuilders/easybuild-easyconfigs#825, looking excellent now, going in Thanks @pescobar! |
No description provided.