-
-
Notifications
You must be signed in to change notification settings - Fork 9.9k
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
Update openssl-smime.pod.in #24052
Update openssl-smime.pod.in #24052
Conversation
remove duplicate entries for -nocerts and -noattr
|
||
=item B<-noattr> | ||
|
||
Don't include any signed attributes when signing. |
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.
Well spotted,
Maybe fix up the other ones so they are just as clear as these ones are.. There needs to at least be a comma in the -nocerts one above.
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.
Could you please also amend the commit message to include the CLA: trivial
annotation?
Have I put it in on the wrong place? |
Yes, it needs to be in the commit message, not just in the pull request description. You can use |
CLA:trivial
1c95cce
to
d9b1df1
Compare
This PR is in a state where it requires action by @openssl/committers but the last update was 30 days ago |
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.
OK with CLA trivial
@paulidale Can you please confirm, you're OK with CLA trivial?
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.
okay with trivial
This pull request is ready to merge |
Merged to all the active branches. Thank you for your contribution. |
Remove duplicate entries for -nocerts and -noattr CLA:trivial Reviewed-by: Paul Dale <ppzgs1@gmail.com> Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from #24052)
Remove duplicate entries for -nocerts and -noattr CLA:trivial Reviewed-by: Paul Dale <ppzgs1@gmail.com> Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from openssl#24052)
remove duplicate entries for -nocerts and -noattr
The manpage for openssl-smime shows two entries for -nocerts and -noattr, respectively. One set of them (l. 198 ff) is quite exhaustive, the second one merely mentions the existence, and thus is redundant.
Checklist
CLA: trivial