search

serde-deprecated / quasi

No longer maintained
38 stars 17 forks source link

`<quote\ expansion` in fingerprints #7

Open ArtemGr opened 9 years ago

ArtemGr commented 9 years ago

When building a serde_macros-dependent code, a string from quasi (https://github.com/erickt/rust-quasi/blob/d9ec71215002076dd0102698b22f78fb8f748e71/quasi/src/lib.rs#L126) ends up in the cargo fingerprint files. For example:

git clone https://github.com/Byron/yup-oauth2
cd yup-oauth2 && cargo build
cat target/debug/.fingerprint/yup-oauth2-*/dep-lib-yup-oauth2

would produce

/tmp/yup-oauth2/tmp/yup-oauth2/target/debug/libyup_oauth2.rlib: src/lib.rs src/device.rs src/refresh.rs src/common.rs src/helper.rs <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion

/tmp/yup-oauth2/target/debug/yup_oauth2.d: src/lib.rs src/device.rs src/refresh.rs src/common.rs src/helper.rs <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion

I wonder if that makes cargo rebuild the project every time (https://github.com/rust-lang/cargo/issues/1215#issuecomment-107750532).

Would appreciate if quasi wouldn't leak these.

ArtemGr commented 9 years ago

Workaround: https://github.com/ArtemGr/cargo/commit/f56369511c2737c5c18a63e382f5bbbae88f8de4