{tools}[dummy/dummy] MLC v3.0#2577
Conversation
|
Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/6601/console for more details). This pull request is now ready for review/testing. Please try and find someone who can tackle this; contact @boegel if you're not sure what to do. |
|
Test report by @boegel |
|
@wpoely86: ... |
| sources = ['mlcv3.0.tgz'] | ||
|
|
||
| sanity_check_paths = { | ||
| 'files': ['Linux/mlc', 'Linux/mlc_avx512', 'mlc_license.txt', 'readme.pdf'], |
There was a problem hiding this comment.
do we really want a license and readme in the sanity check?
There was a problem hiding this comment.
well, they're part of the tarball being unpacked... if they're not there, soemthing is wrong
and we do this in other places too, license and documentation are important part of an installation?
|
Going in, thanks for the review @wpoely86! |
(created using
eb --new-pr)