crepererum commented on PR #8558:
URL: https://github.com/apache/arrow-rs/pull/8558#issuecomment-3376366349

   This currently is only semi-effective until 
https://github.com/BurntSushi/rust-csv/pull/403 is merged and released. But if 
we try this upstream change by adding the following to `Cargo.toml`:
   
   ```toml
   [patch.crates-io]
   csv = {git = "https://github.com/paolobarbolini/rust-csv.git";, branch = 
"switch-to-serde-core"}
   ```
   
   and then we get all crates for a full-featured `arrow` (but without dev 
dependencies because `criterion` uses derive-based serialization):
   
   ```console
   $ cargo tree -p arrow --all-features -e=no-dev
   ```
   
   then there's no `serde_derive` left and no `syn` and no `proc-macro2`. So I 
would call this a win.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to