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

Bump euclid and plane-split. #2882

Closed
wants to merge 1 commit into from
Closed

Conversation

@BorisChiou
Copy link

BorisChiou commented Jul 10, 2018

We bump euclid for direct-composition, examples, webrender,
webrender_api, and wrench.


This change is Reviewable

@BorisChiou
Copy link
Author

BorisChiou commented Jul 10, 2018

I notice that only examples, webrender, webrender_api, and wrench are used in Gecko, and webrender_bindings are not in this repo (which is in Gecko, I guess). That's why I didn't update euclid for pathfinder.

@BorisChiou
Copy link
Author

BorisChiou commented Jul 10, 2018

r? @nical

@BorisChiou
Copy link
Author

BorisChiou commented Jul 10, 2018

OK.. taskclaster said I have to update euclid for pathfinder and lyon_geom.

@nical
Copy link
Collaborator

nical commented Jul 10, 2018

@BorisChiou ouch I'm sorry you are going through all of this for a non-breaking change. the latest version of lyon_geom already depends on euclid 0.18.0 so if you update pathfinder to that version it should be good (pathfinder being the only crate that depends on lyon_geom). Hopefully the things in lyon_geom that pathfinder use haven't changed too much since the version pathfinder is using. Let me know if you run into issues updating it (I maintain lyon_geom).
Edit: actually pathfinder needs to be updated to lyon_path 0.11 as well But I don't think that one changed in any way that can break pathfinder.

@bors-servo
Copy link
Contributor

bors-servo commented Jul 11, 2018

The latest upstream changes (presumably #2878) made this pull request unmergeable. Please resolve the merge conflicts.

We bump euclid for direct-composition, examples, webrender,
webrender_api, and wrench. Also, bump plane-split so we could
drop the old version of euclid.
@BorisChiou BorisChiou force-pushed the BorisChiou:bump/euclid branch from c045acc to 8feaed7 Jul 17, 2018
@BorisChiou
Copy link
Author

BorisChiou commented Jul 17, 2018

OK, let's try again

@BorisChiou
Copy link
Author

BorisChiou commented Jul 17, 2018

r? @nical

@BorisChiou
Copy link
Author

BorisChiou commented Jul 17, 2018

cargo test runs successfully on my laptop.

@Eijebong
Copy link
Member

Eijebong commented Jul 17, 2018

@BorisChiou This is superseded by #2904
Sorry, didn't see this one before

@BorisChiou
Copy link
Author

BorisChiou commented Jul 17, 2018

It's ok. I will close this. Thanks for bumping the version. :)

@BorisChiou BorisChiou closed this Jul 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.