Skip to content
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

[curl] Don't try to include OpenSSL headers when OpenSSL backends aren't selected. #24766

Merged
merged 1 commit into from
May 18, 2022

Conversation

BillyONeal
Copy link
Member

@BillyONeal BillyONeal commented May 18, 2022

Fixes regression introduced in #24348

/cc @elms

Thanks to r41nm4ker on the vcpkg Discord for reporting this problem.

@BillyONeal BillyONeal added category:port-bug The issue is with a library, which is something the port should already support info:internal This PR or Issue was filed by the vcpkg team. labels May 18, 2022
@BillyONeal
Copy link
Member Author

This actually makes the patch match upstream https://github.com/curl/curl/pull/8864/files#diff-46a15cd5e1ed3bea97b13d252b6b1f1adc1754a2ed5154a724939b6a1bd06b18R41 so I'm going ahead with it without other contributor input to unbreak curl

@LilyWangLL LilyWangLL assigned LilyWangLL and unassigned Adela0814 May 18, 2022
@BillyONeal BillyONeal merged commit 0a6002e into microsoft:master May 18, 2022
@BillyONeal BillyONeal deleted the 2022-05-17-curl-fix branch May 18, 2022 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-bug The issue is with a library, which is something the port should already support info:internal This PR or Issue was filed by the vcpkg team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants