Skip to content

macOS support for font loading #830

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

Merged
merged 1 commit into from
Feb 7, 2017
Merged

Conversation

jrmuizel
Copy link
Collaborator

@jrmuizel jrmuizel commented Feb 4, 2017

This change is Reviewable

@glennw
Copy link
Member

glennw commented Feb 5, 2017

Looks like there is a compile error on Linux CI. I guess that extern crate should be mac only?

@matprec
Copy link
Contributor

matprec commented Feb 6, 2017

Yup, looking into this

@matprec
Copy link
Contributor

matprec commented Feb 6, 2017

@jrmuizel Can you please confirm that version "0.1.3" is working on macos? Does on my Ubuntu VM.

@jrmuizel
Copy link
Collaborator Author

jrmuizel commented Feb 6, 2017

@MSleepyPanda It works on macOS

@matprec
Copy link
Contributor

matprec commented Feb 6, 2017

sigh please update to 0.1.4
Seriously need to setup ci for font-loader.

@glennw
Copy link
Member

glennw commented Feb 6, 2017

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit f6af0ad has been approved by glennw

@bors-servo
Copy link
Contributor

⌛ Testing commit f6af0ad with merge 437bf5d...

bors-servo pushed a commit that referenced this pull request Feb 7, 2017
macOS support for font loading

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/830)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

☀️ Test successful - status-travis
Approved by: glennw
Pushing 437bf5d to master...

@bors-servo bors-servo merged commit f6af0ad into servo:master Feb 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants