add easyblock for Mathematica#240
Conversation
There was a problem hiding this comment.
Exage-rated style here ;-) otherwise reviewed and good.
|
Fix style remark, thanks for the review @fgeorgatos The easyconfig PR still needs to be reviewed though, see easybuilders/easybuild-easyconfigs#394. We haven't been able to test the Mathematica installation on our end yet because of license server woes, so I'll hold off merging this in until our installation has been actually tested... |
|
Tested installation obtained with this easyblock, works, as expected. @fgeorgatos: Please also review the easyconfig PR (easybuilders/easybuild-easyconfigs#394) so this can be merged in. |
There was a problem hiding this comment.
I would rename this to default_qa_install_path or even qa_install_path;
otherwise, this PR is fine and ready for test/merge.
No description provided.