search

jmcnamara / rust_xlsxwriter

A Rust library for creating Excel XLSX files.
https://crates.io/crates/rust_xlsxwriter
Apache License 2.0
316 stars 25 forks source link

Remove dependency on Zstd (from ZipWriter) for easier wasm setup #11

Closed rRaDuCaN closed 1 year ago

rRaDuCaN commented 1 year ago

Provide a lighter version of the lib without any c lib dependencies, for a possibly, web assembly integration. Zstd is driving me crazy with wasm setup.

jmcnamara commented 1 year ago

I've pushed a change to main to reduce the zipwriter dependencies down to just "deflate" and effectively removing the "zstd" dependency. Can you try that and let me know how you get on. That will still have a C dependency but may be easier to get to compile.

jmcnamara commented 1 year ago

And just in case it isn't clear you can test it by changing your dependency to this:

[dependencies]
rust_xlsxwriter = {git = "https://github.com/jmcnamara/rust_xlsxwriter"}
jmcnamara commented 1 year ago

See also the changes in #12 which might help.

jmcnamara commented 1 year ago

Did this change help/fix your issue?

rRaDuCaN commented 1 year ago

@jmcnamara , works like a charm, thank you so much.

jmcnamara commented 1 year ago

@rRaDuCaN That's great. Thanks for letting me know. I'll push out the change shortly to crates.io in a dot release.

Btw, would you care to share the step you did to did to get it working with wasm. I plan to add a section about that in the User Guide in the next few week so any workaround or things to look out for, for the end user, would be helpful.

jmcnamara commented 1 year ago

Thanks for the suggestion.

Fixed in version 0.12.1. Closing.