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

Get `font-kit` compiling under wasm #67

Merged
merged 1 commit into from May 28, 2019
Merged

Get `font-kit` compiling under wasm #67

merged 1 commit into from May 28, 2019

Conversation

@pcwalton
Copy link
Collaborator

pcwalton commented May 27, 2019

No description provided.

@pcwalton pcwalton merged commit 693438d into master May 28, 2019
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details
@pcwalton pcwalton deleted the wasm branch May 28, 2019
@RazrFalcon

This comment has been minimized.

Copy link
Collaborator

RazrFalcon commented on src/sources/mod.rs in cdd02e0 May 29, 2019

Shouldn't all three modules be under the cfg attribute? Because currently, only the fs mod will be disabled.

This comment has been minimized.

Copy link
Collaborator Author

pcwalton replied May 29, 2019

mem should be OK under wasm (in fact, it's the only source that works). Maybe I should add a newline there to make it clear.

This comment has been minimized.

Copy link
Collaborator

RazrFalcon replied May 29, 2019

Yes, a new line will be great, because it's a bit misleading right now.

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

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