{tools}LSMS-3_rev237-foss-2016a#4335
Conversation
next step chown root:root /apps/all/Singularity/*/etc/singularity/singularity.conf chown root:root /apps/all/Singularity/*/libexec/singularity/sexec-suid chmod +s /apps/all/Singularity/*/libexec/singularity/sexec-suid
…figs into singularity_2.2.1
…igs into singularity_2.2.1
|
@foxik0070 I've changed the target branch to |
|
|
||
| files_to_copy = ['bin'] | ||
|
|
||
| buildopts = '-j 1' |
There was a problem hiding this comment.
is this strictly required?
if so, use
parallel = 1|
|
||
| dependencies = [ | ||
| ('HDF5', '1.8.16'), | ||
| ] |
There was a problem hiding this comment.
please move this up, below patches
…= 1 && move dependencies, below patches
| @@ -0,0 +1,106 @@ | |||
| --- LSMS_3_rev237/CBLAS/Makefile 2012-02-09 20:17:00.000000000 +0100 | |||
There was a problem hiding this comment.
@foxik0070 please include a description of the patch + link to where it comes from and/or author
|
|
||
| checksums = ['8c864c9223b93488663815557e61cd0b'] | ||
|
|
||
| patches = ['LSMS-Makefiles.patch'] |
There was a problem hiding this comment.
@foxik0070 please rename the patch to include the version, something like
patches = ['LSMS-3_rev237_fix-Makefiles.patch']|
@boegel OK, I changed buildopts and move dependency |
|
Test report by @boegel |
|
|
||
| checksums = ['8c864c9223b93488663815557e61cd0b'] | ||
|
|
||
| patches = ['LSMS-3_rev237_fix-Makefiles.patch'] |
There was a problem hiding this comment.
@foxik0070 please avoid hardcoding the version, use the %(version)s template instead:
patches = ['LSMS-%(version)s_fix-Makefiles.patch']|
Test report by @boegel |
…-%(version)s_fix-Makefiles.patch
|
lgtm |
|
Going in, thanks @foxik0070! |
|
There is something really weird with this PR: it deleted |
|
@wpoely86 singularity will be added separately |
|
Yes, but that's not the point. It should show up in the diff here. |
|
File was deleted, diff not existing |
|
A, now I understand what happened. You used the same branch as for #4250. Please don't do that again @foxik0070. Now it appears that @boegel merged @boegel We should keep an eye open for situations like this. |
|
It was a mistake |
|
Don't worry about it @foxik0070 |
Test result
Successfully built /apps/easybuild/easyconfigs-it4i/l/LSMS/LSMS-3_rev237-foss-2016a.eb
Overview of tested easyconfigs (in order)
Time info
EasyBuild info
System info
List of loaded modules
Environment