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

Update curl to 7.74.0, libssh2 to 1.9.0, libs3 to match. #336

Merged
merged 5 commits into from
Apr 21, 2021

Conversation

ned14
Copy link

@ned14 ned14 commented Jan 4, 2021

  • I've followed this guide
    step by step carefully. Yes

  • I've checked this Git style guide. Yes

  • I've checked this CMake style guide. Yes

  • My change will work with CMake 3.2 (minimum requirement for Hunter). Yes

  • I will try to keep this pull request as small as possible and will try not to mix unrelated features. Yes

Note that some of the build matrix has been disabled, as upstream OpenSSL dependency doesn't work with them.

@@ -46,7 +46,7 @@ hunter_default_version(BoringSSL VERSION 0.0.0-0f5ecd3a8-p0)
hunter_default_version(Box2D VERSION 2.3.1-p0)
hunter_default_version(CLAPACK VERSION 3.2.1)
hunter_default_version(CLI11 VERSION 1.8.0)
hunter_default_version(CURL VERSION 7.60.0-p2)
hunter_default_version(CURL VERSION 7.74.0-p0)
Copy link
Member

Choose a reason for hiding this comment

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

These versions don't exist yet - please add them to the hunter.cmake files for each project.

@rbsheth
Copy link
Member

rbsheth commented Jan 4, 2021

@ned14 Looks like curl and s3 failed all tests - can you check on them?

@ned14 ned14 force-pushed the master branch 6 times, most recently from 2cb4875 to 8155b4d Compare January 6, 2021 15:15
@ned14 ned14 force-pushed the master branch 2 times, most recently from 42da327 to 503785c Compare January 22, 2021 16:31
@rbsheth
Copy link
Member

rbsheth commented Jan 22, 2021

@ned14 Can you restore the libs3 change and I will merge. Thanks!

@rbsheth
Copy link
Member

rbsheth commented Jan 29, 2021

@ned14 Can you restore the libs3 change and I will merge. Thanks!

@ned14 ping! 😄

@ned14
Copy link
Author

ned14 commented Feb 1, 2021

@ned14 Can you restore the libs3 change and I will merge. Thanks!

@ned14 ping! 😄

Sorry about missing your request last week. I had a lot of spinning plates recently.

@rbsheth
Copy link
Member

rbsheth commented Feb 8, 2021

@ned14 Can you restore the libs3 change and I will merge. Thanks!

@ned14 ping! 😄

Sorry about missing your request last week. I had a lot of spinning plates recently.

No worries, thanks!

cmake/projects/CURL/ci/matrix.json Outdated Show resolved Hide resolved
cmake/projects/CURL/hunter.cmake Outdated Show resolved Hide resolved
examples/CURL/CMakeLists.txt Outdated Show resolved Hide resolved
@rbsheth
Copy link
Member

rbsheth commented Mar 16, 2021

@rbsheth rbsheth merged commit 2e25878 into cpp-pm:master Apr 21, 2021
@rbsheth
Copy link
Member

rbsheth commented Apr 21, 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