-
Notifications
You must be signed in to change notification settings - Fork 14
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
error when target is i686-pc-windows-msvc #18
Comments
@QiWei thanks for making an issue! I've started looking into this but don't have a good windows setup to debug this on at the moment outside of CI. I've opened an issue in the libdeflater crate, which is where the fix will likely need to happen. In the meantime you can disable default features for gzp and set the "deflate_default,snappy_default" features manually and everything should "just work", with a bit of a performance hit because libdeflater is so fast (but it is slower on 32 bit...). Passing CI for i686-pc-windows-msvc with the above mentioned features: https://github.com/sstadick/gzp/runs/3569445386?check_suite_focus=true |
@sstadick hit this recently for the same target, and your suggestion fixed it! Perhaps this should be added to the README? |
Link added in the README! Thanks for the call out 👍 An issue to keep half an eye is this one: #44 zune should be able to replace libdeflate, and be as or more performant, while being pure rust. I haven't had time to get into it yet, but it seems very promising. Currently the impl is asymmetrical and I may not support it till it can both compress and decompress. |
The text was updated successfully, but these errors were encountered: