-
-
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
Fix compilation on Windows using icc #23415
Conversation
@UlrichKamke can you confirm that this patch fixes the build and that the tests still pass? |
The parameter list for CRYPTO_DOWN_REF for the icc on windows build was incorrect. This issue was introduced by 99fd5b2 Fixes openssl#23414
00e026c
to
82f9682
Compare
This pull request is ready to merge |
Hello Matt, why do you merged your fix from #23414 not in the release 3.2.1? So we could use an official stable version 3.2.1. Your patch 23414 works for us. |
We were in code freeze from last week due to the release. This fix didn't hit "ready-to-merge" status until too late, so missed the 3.2.1 release. It will be merged soon and will be available in 3.2.2. |
Pushed to master/3.2. |
The parameter list for CRYPTO_DOWN_REF for the icc on windows build was incorrect. This issue was introduced by 99fd5b2 Fixes openssl#23414 Reviewed-by: Richard Levitte <levitte@openssl.org> Reviewed-by: Hugo Landau <hlandau@openssl.org> (Merged from openssl#23415)
The parameter list for CRYPTO_DOWN_REF for the icc on windows build was incorrect.
This issue was introduced by 99fd5b2
Note I do not have access to this environment to test this...
Fixes #23414