Skip to content

cmake: don't require OpenSSL if CMAKE_USE_OPENSSL=OFF#3001

Closed
slodki wants to merge 1 commit intocurl:masterfrom
slodki:patch-1
Closed

cmake: don't require OpenSSL if CMAKE_USE_OPENSSL=OFF#3001
slodki wants to merge 1 commit intocurl:masterfrom
slodki:patch-1

Conversation

@slodki
Copy link
Copy Markdown
Contributor

@slodki slodki commented Sep 15, 2018

User must have OpenSSL installed even if not used by libcurl at all since 7.61.1 release.
Broken at 7867aaa
More complex solution in #2849 (including this fix)

User must have OpenSSL installed even if not used by libcurl at all since 7.61.1 release.
Broken at 7867aaa
Copy link
Copy Markdown
Contributor

@snikulov snikulov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bagder
Copy link
Copy Markdown
Member

bagder commented Sep 18, 2018

Thanks!

@bagder bagder closed this in 71b3c67 Sep 18, 2018
@lock lock bot locked as resolved and limited conversation to collaborators Dec 17, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Development

Successfully merging this pull request may close these issues.

3 participants