Compile serde_derive separately from serde #866
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When
serde
'sderive
feature is used,serde_derive
must be compiled beforeserde
can be, asserde
with that feature has aserde_derive
dependency.As of serde 1.0.186, this issue can be avoided by adding a separate
serde_derive
dependency due to the fact that serde 1.0.186 has a never-applicable dependency onserde_derive
, which ensures that there is no incompatible version ofserde_derive
in a program (serde-rs/serde#2588).Because MSRV being set to 1.56, it's not possible to use the
dep:
syntax in features, soserde
crate needs to be renamed.dep:
syntax was added in Rust 1.60 if we want to bump MSRV.This should improve compilation times of programs that use
url
with itsserde
feature, provided it doesn't have other crates that useserde
with itsderive
feature.