-
Notifications
You must be signed in to change notification settings - Fork 203
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
allow setting post message from easyconfig file #542
Comments
+1 :) Have the case for LikWid where a setuid procedure is required for the daemon. |
@Falkor What's the use case you have in mind? |
I just updated the easybuild recipes for Likwid that relies on the There are 3 ways to do that (setuid / setgid or libcap methods), only the first one actually works, but in all cases it would be worth to give the [sudo] commands to perform to the user. postinstallcmds = [
'echo "IMPORTANT: you will have to manually apply the following changes:"',
'echo " sudo chown root:root \$EBROOTLIKWID/sbin/likwid-accessD && sudo chmod u+s \$EBROOTLIKWID/sbin/likwid-accessD"'
] but that hides it in the build logs as they are not exposed to the user on the regular |
@victorusu was looking for suggestions for contributions, maybe this is an easy one to tackle... |
I have implemented a solution with following usage: postinstallmsgs = ["This is post install message 1", "This is post install message 2"]
|
In some cases, it would be interesting to allow setting the post message that is printed at the end of the installation process in the easyconfig file, e.g. for issuing warnings in case like
bbcp
which requires certain ports to be open for it to work properly.The
easyblock
module already supports this via theself.postmsg
class variable.The text was updated successfully, but these errors were encountered: