-
-
Notifications
You must be signed in to change notification settings - Fork 9.8k
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
Fix the recent daily runchecker CI failures related to QUIC #21677
Conversation
Fixes openssl#21624 With OPENSSL_NO_POSIX_IO or OPENSSL_NO_SOCK the function wait_until_sock_readable() does not exist.
as both algorithms are really needed. Fixes openssl#21625
Also do not build quicserver with no-stdio as it is a test utility and tests are disabled with no-stdio anyway.
Revert "quicapitest.c: Do not use undefined function" Define empty wait_until_sock_readable() when building with no-posix-io.
on: | ||
schedule: | ||
- cron: '0 6 * * *' | ||
on: [pull_request] |
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.
Is this change just temporary while the PR is being worked on?
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.
Yes - it is in a commit marked with drop!
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
Note for merging: Do not forget to drop the commit marked drop! |
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. |
Merged to master branch. Thank you for the reviews. |
Fixes #21624 With OPENSSL_NO_POSIX_IO or OPENSSL_NO_SOCK the function wait_until_sock_readable() currently does not exist. Define empty wait_until_sock_readable() when building with no-posix-io. Reviewed-by: Hugo Landau <hlandau@openssl.org> Reviewed-by: Tom Cosgrove <tom.cosgrove@arm.com> (Merged from #21677)
Reviewed-by: Hugo Landau <hlandau@openssl.org> Reviewed-by: Tom Cosgrove <tom.cosgrove@arm.com> (Merged from #21677)
Fixes #21623 #21624 #21625 #21626