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

added Windows SSL support for CMake build #399

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@snikulov
Member

snikulov commented Aug 25, 2015

Re-enabled Windows SSL support for CMake build.

Tested with MSVC 2013/2015, MinGW 5.1.0.

c:\WORK\GitHub\curl\mingw\src>curl --version
curl 7.45.0-DEV (Windows) libcurl/7.45.0-DEV WinSSL
Protocols: dict file ftp ftps gopher http https imap imaps pop3 pop3s rtsp smtp smtps telnet tftp
Features: SSPI Kerberos SPNEGO NTLM SSL

@snikulov

This comment has been minimized.

Member

snikulov commented Aug 25, 2015

@bradking Could you please review and comment? Thank you.

@bradking

This comment has been minimized.

Contributor

bradking commented Aug 25, 2015

Good start. If HAVE_SCHANNEL_H succeeds but HAVE_SSPI_H fails then one will get compilation errors with the current logic. We cannot allow the configured header to define USE_SCHANNEL but not USE_WINDOWS_SSPI. Please update the logic accordingly.

Style nit: the CMakeLists.txt file currently uses 2-space indentation but the patch adds a block using 4-space indentation.

@snikulov

This comment has been minimized.

Member

snikulov commented Aug 25, 2015

@bradking fixed both comments 5d1581c4b89b8b334de9dad5883e91ef57d88441

@bradking

This comment has been minimized.

Contributor

bradking commented Aug 25, 2015

LGTM. Please squash down to one commit.

@snikulov

This comment has been minimized.

Member

snikulov commented Aug 25, 2015

@bradking Done.
@bagder Could you please review and merge?
Thank you.

@bagder bagder closed this in 2132708 Aug 25, 2015

jgsogo added a commit to jgsogo/curl that referenced this pull request Oct 19, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment