Skip to content

Add flate2/libc feature for flate2-zlib #1037

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

Closed
wants to merge 2 commits into from
Closed

Add flate2/libc feature for flate2-zlib #1037

wants to merge 2 commits into from

Conversation

peterschwarz
Copy link

Due to a change in flate2 1.0.10, when "default-features" is false, it is now required to specfy "libc" when specifying the "miniz-sys" feature, in order to compile correctly.

See the comments in rust-lang/flate2-rs#205

Due to a change in flate2 1.0.10, when "default-features" is false, it
is now required to specfy "libc" when specifying the "miniz-sys"
feature, in order to compile correctly.

Signed-off-by: Peter Schwarz <[email protected]>
@JohnTitor
Copy link
Member

cc #1036

@fafhrd91
Copy link
Member

@peterschwarz could you update changes.md as well

@peterschwarz
Copy link
Author

@fafhrd91 Does that go under the 0.2.9 section?

@fafhrd91
Copy link
Member

fafhrd91 commented Aug 14, 2019

0.2.9 is released already

@peterschwarz
Copy link
Author

Ok, little help then (this is my first PR on this repo): what's the format for that file for new changes?

@peterschwarz
Copy link
Author

Nevermind, found an example.

@peterschwarz
Copy link
Author

Looks like this error will be taken care of upstream: rust-lang/flate2-rs#206. Perhaps this should be closed, then.

@JohnTitor
Copy link
Member

The building issue has been fixed in flate2 v1.0.11

@fafhrd91
Copy link
Member

Very good

@fafhrd91 fafhrd91 closed this Aug 15, 2019
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.

3 participants