conda-forge / curl-feedstock

A conda-smithy repository for curl.
BSD 3-Clause "New" or "Revised" License
3 stars 42 forks source link

Update to 7.49.0 #5

Closed jakirkham closed 8 years ago

jakirkham commented 8 years ago

Updates to curl 7.49.0. I don't think we need to do pinnings based on what curl says about ABI compatibility.

conda-forge-linter commented 8 years ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

jakirkham commented 8 years ago

Seems we require perl to run the curl's tests. This is being addressed in PR ( https://github.com/conda-forge/curl-feedstock/pull/6 ).

jakirkham commented 8 years ago

As PR ( https://github.com/conda-forge/curl-feedstock/pull/6 ) is merged, this has been rebased on top of master. Assuming this passes, it should be fine to merge.

jakirkham commented 8 years ago

Hitting two failures. See the log. Snippet of errors shown below.

TESTFAIL: These test cases failed: 1139 1140

Turns out this came up on the curl mailing list recently. See the referenced issue ( https://github.com/curl/curl/issues/814 ). Appears the solution is to just go to 7.49.1.

jakirkham commented 8 years ago

Closing out in favor of PR ( https://github.com/conda-forge/curl-feedstock/pull/7 ). Basically, we are leapfrogging this release to fix some test failures.