Skip to content

Fix --add-seccomp-fd argument name in usage#482

Merged
smcv merged 1 commit intocontainers:mainfrom
rusty-snake:fix-add-seccomp-fd
Mar 22, 2022
Merged

Fix --add-seccomp-fd argument name in usage#482
smcv merged 1 commit intocontainers:mainfrom
rusty-snake:fix-add-seccomp-fd

Conversation

@rusty-snake
Copy link
Copy Markdown
Contributor

@rusty-snake rusty-snake commented Mar 19, 2022

--help shows --add-seccomp instead of --add-seccomp-fd which is the
correct argument.

@smcv
Copy link
Copy Markdown
Collaborator

smcv commented Mar 22, 2022

Please could you add a Signed-off-by to indicate your acceptance of the Developer Certificate of Origin? (More details: https://github.com/containers/common/blob/main/CONTRIBUTING.md#sign-your-prs)

(@cgwalters: Do we require DCO on trivial changes like this?)

smcv
smcv approved these changes Mar 22, 2022
View reviewed changes
Copy link
Copy Markdown
Collaborator

@smcv smcv left a comment

Choose a reason for hiding this comment

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

The actual change looks fine when the DCO thing is resolved. Thanks!

@rusty-snake
Fix --add-seccomp-fd argument name in usage
0affc24 
--help shows --add-seccomp instead of --add-seccomp-fd which is the
correct argument.

Signed-off-by: rusty-snake <41237666+rusty-snake@users.noreply.github.com>
@smcv smcv merged commit 920c88e into containers:main Mar 22, 2022
@rusty-snake rusty-snake deleted the fix-add-seccomp-fd branch March 22, 2022 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@smcv smcv smcv approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rusty-snake @smcv