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

man: minor clarifications to man pages #3445

Merged

Conversation

jsquyres
Copy link
Contributor

@jsquyres jsquyres commented Jun 2, 2020

Add verbiage to the man pages clarifying that the files/directories in the lists given to options such as --private-bin must be relative to the directory that is being limited (e.g., --private-bin requires a list of files/directories that are relative to /bin).

Signed-off-by: Jeff Squyres jeff@squyres.com

I made this PR because it took me a while to figure out why I couldn't properly restrict things with --private-opt. Adding this verbiage will hopefully save another newbie like me some time in the future!

Add verbiage to the man pages clarifying that the files/directories in
the lists given to options such as --private-bin must be relative to
the directory that is being limited (e.g., --private-opt requires a
list of files/directories that are relative to /opt).

Signed-off-by: Jeff Squyres <jeff@squyres.com>
@jsquyres jsquyres force-pushed the pr/minor-man-page-clarifications branch from 9fc8510 to ef9bd32 Compare June 2, 2020 23:20
@jsquyres
Copy link
Contributor Author

jsquyres commented Jun 2, 2020

Updated.

@Fred-Barclay
Copy link
Collaborator

@jsquyres @rusty-snake all good? I'm good with merging.

@jsquyres
Copy link
Contributor Author

jsquyres commented Jun 4, 2020

I'm happy with it!

@rusty-snake rusty-snake merged commit 40d3604 into netblue30:master Jun 4, 2020
@rusty-snake
Copy link
Collaborator

Merged, Thanks.

@jsquyres jsquyres deleted the pr/minor-man-page-clarifications branch June 4, 2020 18:53
@matu3ba matu3ba mentioned this pull request Oct 7, 2021
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.

3 participants