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

Make FindCURL ready for vcpkg #8295

Merged
merged 1 commit into from
Jul 26, 2019
Merged

Conversation

adrido
Copy link
Contributor

@adrido adrido commented Feb 27, 2019

This allows to find and use curl with the vcpkg toolchain file.

There will be another PR regarding vcpkg on Windows, but this is kind of urgent: Never ever copy a file out of the Windows directory and redistribute them!
That might not work from technical aspects (x32 vs x64), it is also very questionable in terms of license.

Last but not least: This also allows to build Minetest on windows with a static linked version of curl.

This allows to find and use curl with the vcpkg toolchain file.
@paramat paramat added the Feature ✨ PRs that add or enhance a feature label Feb 28, 2019
@sfan5
Copy link
Member

sfan5 commented Jul 26, 2019

👍

@MoNTE48
Copy link
Contributor

MoNTE48 commented Jul 27, 2019

0 changed files...
af72876

@SmallJoker
Copy link
Member

Included in bd6f1cc @adrido.
Interestingly this PR did not show up rebase conflicts due to the identical code changes.

@adrido
Copy link
Contributor Author

adrido commented Jul 28, 2019

Sorry, my fault. Should have closed this PR.

@adrido adrido deleted the fix-find-curl branch July 28, 2019 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature ✨ PRs that add or enhance a feature >= Two approvals ✅ ✅
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants