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.

Sign up for GitHub

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

Jump to bottom

support for emscripten #172

Closed
wants to merge 4 commits into from
Closed

support for emscripten #172

wants to merge 4 commits into from

Conversation

DoDoENT
Copy link
Contributor

@DoDoENT DoDoENT commented Aug 22, 2019

  • allow emscripten as supported platform
  • skip libcxx detection with emscripten, as it will fail with em++ - simply set libcxx, as it's the only possible option at the moment

@czoido czoido added this to the 0.16 milestone Jan 16, 2020
@czoido czoido mentioned this pull request Jul 10, 2020
Merged
@czoido
Copy link
Contributor

czoido commented Sep 3, 2020

Hi @DoDoENT,
Sorry for the late response and thanks a lot for the contribution.
I think that this should be addressed by these two PR that have already been merged:

I'm closing this PR, but please report if you have any issues.

@czoido czoido closed this Sep 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.16
Development

Successfully merging this pull request may close these issues.
2 participants
@DoDoENT @czoido