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
There is no -signreq option in CA.pl #11876
Conversation
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.
LGTM. And I agree with the trivial designation.
Unfortunately for 1.1.1 there should be a separate PR because the -signreq is mentioned at one additional place there. |
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.
Approved. I agree this is trivial.
This pull request is ready to merge |
CLA: trivial Reviewed-by: Matt Caswell <matt@openssl.org> Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from #11876)
Merged to master, would you like to create a PR for the 1.1.1 branch? |
@t8m is it normal to close a branch with un-merged commits? |
It is actually merged, the e1c6f76 is your commit, rebased and with the commit message adjusted so it contains the reviewed-by and link to the PR. |
yes, because the master source for openssl is not on github, but on an openssl server which the pushes to github as a read-only copy. |
CLA: trivial
Checklist