Skip to content
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

{tools}Singularity-2.2.1-GCC-6.3.0 added #4350

Merged

Conversation

foxik0070
Copy link
Contributor

next steps after instalations

# INSTALATION_PATH=your_instalation_path
# chown root:root $INSTALATION_PATH/Singularity/*/etc/singularity/singularity.conf
# chown root:root $INSTALATION_PATH/Singularity/*/libexec/singularity/sexec-suid
# chmod +s $INSTALATION_PATH/Singularity/*/libexec/singularity/sexec-suid

@foxik0070
Copy link
Contributor Author

@wpoely86 @boegel I verry sorry (#4335)

@wpoely86
Copy link
Member

Can you add those next steps in the easyconfig itself as comments?

@foxik0070
Copy link
Contributor Author

it would be possible to somehow insert a provision to eb?

@wpoely86
Copy link
Member

@foxik0070 what do you mean?

@foxik0070
Copy link
Contributor Author

@wpoely86 postinstall script or commands for:

# INSTALATION_PATH=your_instalation_path
# chown root:root $INSTALATION_PATH/Singularity/*/etc/singularity/singularity.conf
# chown root:root $INSTALATION_PATH/Singularity/*/libexec/singularity/sexec-suid
# chmod +s $INSTALATION_PATH/Singularity/*/libexec/singularity/sexec-suid

@wpoely86
Copy link
Member

You have postinstallcmds for that but as you need root or sudo rights, it will probably fail. So you're probably better off by adding it in comments.

@boegel boegel modified the milestones: 3.2.0, 3.3.0 Apr 26, 2017
@vanzod
Copy link
Member

vanzod commented May 17, 2017

I tested and it seems ok here. Any issue that prevents it from merging?

@boegel
Copy link
Member

boegel commented May 18, 2017

@vanzod Well, I guess it can go in, it's just that you need to make some manual changes afterwards...

@boegel
Copy link
Member

boegel commented May 18, 2017

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2568.golett.os - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/4d5398402b8257d00efbc91c799fd418 for a full test report.

@boegel
Copy link
Member

boegel commented May 18, 2017

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2017.delcatty.os - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/d198c1c8169561221c76ae205d126be3 for a full test report.

@boegel
Copy link
Member

boegel commented May 18, 2017

lgtm

@boegel
Copy link
Member

boegel commented May 18, 2017

Going in, thanks @foxik0070!

@boegel boegel merged commit 19cfa4b into easybuilders:develop May 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants