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

Enable linking against static cURL on Windows #2514

Merged
merged 2 commits into from
Feb 20, 2021
Merged

Enable linking against static cURL on Windows #2514

merged 2 commits into from
Feb 20, 2021

Conversation

mloskot
Copy link
Member

@mloskot mloskot commented Feb 3, 2021

This is preliminary fix to allow linking the library
as well as its dependants (e.g. PROJ apps) against cURL
built as static library on Windows with support of native
Windows TLS/SSL.

On Windows, such static cURL (and its dependants) requires linking
against Winsock, CryptoAPI and other networking libaries.


This patch is a minimal version confirm to work with cURL 7.75.0 built as SChannel mode (old name WinSSL, see curl/curl#3504), that is MSVC outputs binaries into:

curl\builds\libcurl-vc14-x86-debug-static-ipv6-sspi-schannel\
curl\builds\libcurl-vc14-x86-release-static-ipv6-sspi-schannel\ 

References

mloskot added a commit to Cadcorp/PROJ that referenced this pull request Feb 5, 2021
…RL 7.75.0

We are going to fix this for the upstream
OSGeo#2514
@kbevers
Copy link
Member

kbevers commented Feb 19, 2021

@mloskot with #2533 are you able to make this work now? It seems a merge conflict has arised. The fix seems trivial but not sure if the rest of the PR is in good shape.

This is preliminary fix to allow linking the library
as well as its dependants (e.g. PROJ apps) against cURL
built as static library on Windows with support of native
Windows TLS/SSL.
On Windows, such static cURL (and its dependants) requires linking
against Winsock, CryptoAPI and other networking libaries.
@mloskot
Copy link
Member Author

mloskot commented Feb 19, 2021

@kbevers Thanks for reminder. I've just updated the PR resolving the conflict.

@mloskot mloskot changed the title WIP: Enable linking against static cURL on Windows Enable linking against static cURL on Windows Feb 19, 2021
@kbevers
Copy link
Member

kbevers commented Feb 19, 2021

I've just updated the PR resolving the conflict.

... and it took me like five minutes to create another conflict :-) At least I know how to fix this one.

@kbevers kbevers added this to the 8.0.0 milestone Feb 20, 2021
@kbevers kbevers merged commit c401528 into OSGeo:master Feb 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants