Skip to content

Comments

man: s_server: fix text repetition in -alpn description#15099

Closed
tomato42 wants to merge 1 commit intoopenssl:masterfrom
tomato42:fix-s_server-man-page-repeat
Closed

man: s_server: fix text repetition in -alpn description#15099
tomato42 wants to merge 1 commit intoopenssl:masterfrom
tomato42:fix-s_server-man-page-repeat

Conversation

@tomato42
Copy link
Contributor

@tomato42 tomato42 commented Apr 30, 2021

Fix the repetition in the description of -alpn option in s_server man page

Checklist
  • documentation is added or updated
  • tests are added or updated

since the paths are different I've filed also a 1.1.1 branch PR in #15098

@t8m t8m added approval: review pending This pull request needs review by a committer branch: master Applies to master branch labels Apr 30, 2021
@beldmit beldmit added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Apr 30, 2021
@beldmit
Copy link
Member

beldmit commented Apr 30, 2021

Approved if test passed

@openssl-machine
Copy link
Collaborator

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set. Please review the updates and set the label manually.

openssl-machine pushed a commit that referenced this pull request May 1, 2021
Reviewed-by: Tomas Mraz <[email protected]>
Reviewed-by: Dmitry Belyavskiy <[email protected]>
(Merged from #15099)
@beldmit
Copy link
Member

beldmit commented May 1, 2021

Merged. Thanks!

@beldmit beldmit closed this May 1, 2021
@tomato42 tomato42 deleted the fix-s_server-man-page-repeat branch May 1, 2021 17:06
devnexen pushed a commit to devnexen/openssl that referenced this pull request Jul 7, 2021
Reviewed-by: Tomas Mraz <[email protected]>
Reviewed-by: Dmitry Belyavskiy <[email protected]>
(Merged from openssl#15099)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: done This pull request has the required number of approvals branch: master Applies to master branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants