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
Check privileges of user #3011
Check privileges of user #3011
Conversation
Check if the script is being run as root or sudo, else exit with message.
Thank you for your submission. It will be reviewed soon and submitted for processing in CI. |
@@ -2,6 +2,13 @@ | |||
|
|||
# auto-update script for rippled daemon | |||
|
|||
# Check for sudo/root permissions | |||
if [ `id -u` -ne 0 ]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's a nit-pick, but other places here use $(cmd)
for subshells, so I guess I would prefer that strictly for consistency (I don't think there is any practical difference in this case, but let me know if I'm wrong...)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No issues. I can change it for consistency.
Thank you for your submission. It will be reviewed soon and submitted for processing in CI. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good - thanks!
Jenkins Build SummaryBuilt from this commit Built at 20190729 - 21:17:08 Test Results
|
Check if the script is being run as root or sudo, else exit with message.