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

feat: enable using the zlib-rs backend through flate2 #292

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

hasenbanck
Copy link

This backend is exposed since flate2 v1.0.29.

This backend is exposed since flate2 v1.0.29.
@hasenbanck
Copy link
Author

  • Build adn test fails, because the new optional feature would be require rustc 1.75 (zlib-rs), where as the pipeline currently uses 1.73.
  • style_and_docs check fails because of files not touched by this PR.
  • check_minimal_versions won't compile, because the test uses "cargo check --all-features", which will break flate2 (flate 2's CI does choose the backends selectively).

Not sure how to progress with this PR under such constraints.

@Pr0methean Pr0methean enabled auto-merge February 25, 2025 21:42
@Pr0methean
Copy link
Member

You'll probably need to increase the MSRV to 1.75 in Cargo.toml, and either (a) update the CI workflow to replace --all-features with the maximal set of compatible subsets, or (b) fix flate2 upstream so that it builds with --all-features.

@Pr0methean Pr0methean disabled auto-merge February 25, 2025 21:59
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