Skip to content
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

Need help with Euclid update #20130

Closed
paulrouget opened this issue Feb 27, 2018 · 7 comments
Closed

Need help with Euclid update #20130

paulrouget opened this issue Feb 27, 2018 · 7 comments

Comments

@paulrouget
Copy link
Contributor

@paulrouget paulrouget commented Feb 27, 2018

I need to move from Euclid 0.16 to Euclid 0.17.

But then I get this error:

the trait `serde::Deserialize<'_>` is not implemented for `euclid::TypedSize2D<u32, XXX>

Where XXX is CSSPixel, DevicePixel and DeviceIndependentPixel.

Adding a derive to these 3 emums doesn't help.

What the right approach here?

@SimonSapin
Copy link
Member

@SimonSapin SimonSapin commented Feb 27, 2018

Euclid made serde an optional dependency. In the Cargo.toml of the crate that has this error, replace euclid = "0.17" with euclid = {version = "0.17", features = ["serde"]}

@paulrouget
Copy link
Contributor Author

@paulrouget paulrouget commented Feb 27, 2018

Thank you!

@paulrouget paulrouget closed this Feb 27, 2018
@Eijebong
Copy link
Member

@Eijebong Eijebong commented Feb 27, 2018

@paulrouget Are you doing the whole euclid bump in servo ? (I'd like to avoid duplicating work :p)

@paulrouget
Copy link
Contributor Author

@paulrouget paulrouget commented Feb 27, 2018

@Eijebong just started, as I need servo/euclid#274 - but if you already have a PR, I'd rather wait.

@Eijebong
Copy link
Member

@Eijebong Eijebong commented Feb 27, 2018

I don't have a PR but we need to update it in servo and every dependency at once. If you don't do it I'll probably do it tonight.

@paulrouget
Copy link
Contributor Author

@paulrouget paulrouget commented Feb 27, 2018

@Eijebong I'll do it. Looks like a rabbit hole though…

@Eijebong
Copy link
Member

@Eijebong Eijebong commented Feb 27, 2018

It is :D Have fun

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.