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

Update euclid. #87

Merged
merged 2 commits into from Jul 23, 2019
Merged

Update euclid. #87

merged 2 commits into from Jul 23, 2019

Conversation

@emilio
Copy link
Member

emilio commented Jul 22, 2019

No description provided.

@emilio
Copy link
Member Author

emilio commented Jul 22, 2019

r? @pcwalton or @jdm (or anyone else)

@RazrFalcon
Copy link
Collaborator

RazrFalcon commented Jul 22, 2019

I've tried to do this yesterday, but lyon_path was still outdated.

@emilio
Copy link
Member Author

emilio commented Jul 22, 2019

@RazrFalcon yeah, it was updated today.

@emilio
Copy link
Member Author

emilio commented Jul 22, 2019

I guess you can also review this @RazrFalcon (if it looks sane to you)

@RazrFalcon
Copy link
Collaborator

RazrFalcon commented Jul 22, 2019

Yep, I will merge this as soon as CI will be fixed.

@jdm
jdm approved these changes Jul 22, 2019
@@ -1,6 +1,6 @@
[package]
name = "font-kit"
version = "0.3.1"
version = "0.4.0"

This comment has been minimized.

@RazrFalcon

RazrFalcon Jul 22, 2019

Collaborator

I think we should keep it for a separate commit.

This comment has been minimized.

@emilio

emilio Jul 22, 2019

Author Member

Why is that?

This comment has been minimized.

@emilio

emilio Jul 22, 2019

Author Member

Well, I can split it in two anyhow.

@emilio
Copy link
Member Author

emilio commented Jul 22, 2019

Fixed the formatting that was making travis fail and a test too.

@emilio
Copy link
Member Author

emilio commented Jul 22, 2019

And pushed the version bump as a separate commit.

@emilio
Copy link
Member Author

emilio commented Jul 22, 2019

I spent a bit more time and fixed the Windows build.

@pcwalton pcwalton merged commit 721df5d into servo:master Jul 23, 2019
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@emilio emilio deleted the emilio:euclidup branch Jul 23, 2019
jrmuizel referenced this pull request in jrmuizel/raqote Jul 23, 2019
This won't build until https://github.com/pcwalton/font-kit/pull/87 is
published, but I expect it to be merged real soon (tm), and this passes tests
locally (with patches applied).

I need this to update euclid in Servo so as to sync style system changes in
Gecko and Servo.
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.