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 cURL's automatic decompression #166

Merged
merged 1 commit into from
Jan 22, 2020
Merged

Enable cURL's automatic decompression #166

merged 1 commit into from
Jan 22, 2020

Conversation

dgilman
Copy link
Contributor

@dgilman dgilman commented Dec 27, 2019

@ryandesign
Copy link
Contributor

Thank you, but could you please adjust the whitespace of your changes? Your changes use 3 spaces for indentation. MacPorts standard coding style is 4 spaces for indentation, but these files still largely use tabs for indentation. So please either use tabs for indentation to match the rest of the file, or use 4 spaces for each indent of your changes, and we can detab the rest of the file later in a separate commit.

@dgilman
Copy link
Contributor Author

dgilman commented Dec 28, 2019

Oops

@raimue
Copy link
Member

raimue commented Dec 28, 2019

This change looks good to me. Thanks!

Please follow CommitMessages for the commit message format. Add a Closes: to the footer with a reference to the Trac ticket.

@dgilman
Copy link
Contributor Author

dgilman commented Jan 15, 2020

Anything else needs to be done to get this across the finish line?

@raimue raimue merged commit 8c41ca7 into macports:master Jan 22, 2020
@raimue
Copy link
Member

raimue commented Jan 22, 2020

As there were no objections, I merged this. Thanks for your contribution!

@jmroot
Copy link
Member

jmroot commented Jan 24, 2020

@raimue
Copy link
Member

raimue commented Jan 26, 2020

Unfortunately, that test output does not tell anything at all about what failed. I guess it needs someone with access to 10.7 to debug that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants