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.

Sign up for GitHub

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

Jump to bottom

main/curl: enable check() #6673

Closed
wants to merge 1 commit into from
Closed

main/curl: enable check() #6673

wants to merge 1 commit into from

Conversation

andypost
Copy link
Contributor

@andypost andypost commented Mar 15, 2019 •
edited
Loading

@andypost andypost added the A-fix Fixes an abuild label Mar 15, 2019
@andypost andypost requested a review from ncopa March 15, 2019 01:07
@andypost andypost mentioned this pull request Mar 15, 2019
Closed
@prspkt
Copy link

prspkt commented Apr 2, 2019

It appears to have been addressed in 7.64.1

@andypost andypost changed the title main/curl: add patch for threaded-resolver main/curl: upgrade to 7.64.1 Apr 6, 2019
@andypost andypost added A-upgrade Upgrades an abuild and removed A-fix Fixes an abuild labels Apr 6, 2019
@andypost
Copy link
Contributor Author

andypost commented Apr 6, 2019

btw it pass tests now

@andypost andypost mentioned this pull request Apr 12, 2019
Closed
@rnalrd
Copy link
Contributor

rnalrd commented Apr 12, 2019

Sorry saw this too late

@rnalrd rnalrd closed this Apr 12, 2019
@andypost
Copy link
Contributor Author

Still makes sense to enable check

@andypost andypost reopened this Apr 12, 2019
@andypost andypost changed the title main/curl: upgrade to 7.64.1 main/curl: enable check() Apr 12, 2019
@algitbot
Copy link

Merged in 726627b by @rnalrd. Thanks for your contribution!

(This pull request has been closed automatically by GitHub PR Closer. If you think that it’s not resolved yet, please add a comment.)

@algitbot algitbot closed this Apr 17, 2019
@andypost andypost deleted the curl branch April 17, 2019 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tcely tcely tcely approved these changes

@ncopa ncopa Awaiting requested review from ncopa

Assignees
No one assigned
Labels
A-upgrade Upgrades an abuild
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@andypost @prspkt @rnalrd @algitbot @tcely