-
-
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
Add include of e_os2.h in quictestlib.c to allow symbol definition consistency. #22179
Conversation
293c9c5
to
587798f
Compare
I have made this change NonStop-specific but I think it might apply to any c99 builds because the unspecified type, |
This passed build/test on our platform. |
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
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.
We have plenty of code already that uses struct timeval which, as far as I know has compiled just fine on NonStop. I don't think this is the correct fix. I have an alternative suggestion in #22183.
Fixes: openssl#22178 Signed-of-by: Randall S. Becker <randall.becker@nexbridge.ca>
587798f
to
71c7128
Compare
I originally called this "Move" instead of "Add" because it was in comparison to the proposed change by @mattcaswell. This just moves where the |
FYI: This change, commit 71c7128, did pass build/test on NonStop with c99. |
This updated version looks much better. |
This pull request is ready to merge |
Pushed. |
Fixes: #22178 Signed-of-by: Randall S. Becker <randall.becker@nexbridge.ca> Reviewed-by: Paul Dale <pauli@openssl.org> Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: Matt Caswell <matt@openssl.org> (Merged from openssl/openssl#22179) Signed-off-by: fly2x <fly2x@hitls.org>
Fixes: #22178
Signed-of-by: Randall S. Becker randall.becker@nexbridge.ca