Skip to content
This repository was archived by the owner on Feb 24, 2020. It is now read-only.

Conversation

@dm0-
Copy link
Contributor

@dm0- dm0- commented Aug 17, 2016

I originally just wanted to fix the caps-retain example typo, but there were also a few inconsistencies and Markdown formatting errors that got fixed in the process.

@euank
Copy link
Member

euank commented Aug 18, 2016

LGTM, cc @joshix

| --- | --- | --- | --- |
| `--caps-remove` | none | capability to remove (example: '--caps-remove=CAP\_SYS\_CHROOT,CAP\_MKNOD') | Capabilities to remove from the process's capabilities bounding set, all others from the default set will be included |
| `--caps-retain` | none | capability to retain (example: '--caps-remove=CAP\_SYS\_ADMIN,CAP\_NET\_ADMIN') | Capabilities to retain in the process's capabilities bounding set, all others will be removed |
| `--caps-remove` | none | capability to remove (ex. `--caps-remove=CAP_SYS_CHROOT,CAP_MKNOD`) | Capabilities to remove from the process's capabilities bounding set, all others from the default set will be included. |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/ex./e.g./ here and elsewhere
s/, all others/; all others/

@jonboulle
Copy link
Contributor

thanks for the patch!

@joshix
Copy link
Contributor

joshix commented Aug 19, 2016

lgtm

@euank euank merged commit c06e72c into rkt:master Aug 19, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants