Skip to content

Conversation

@tiran
Copy link
Member

@tirantiran commented Jan 11, 2022

Co-authored-by: Ethan Smith ethan@ethanhs.me
Signed-off-by: Christian Heimes christian@python.org

https://bugs.python.org/issue40280

Co-authored-by: Ethan Smith <ethan@ethanhs.me> Signed-off-by: Christian Heimes <christian@python.org>
@tirantiranforce-pushed the bpo-40280-emscripten-targets branch from a1aab67 to b5b4504CompareJanuary 11, 2022 23:46
@brettcannonbrettcannon self-requested a review January 12, 2022 01:08
Copy link
Member

@emmatypingemmatyping left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two suggestions but otherwise this looks good! It is a bit unfortunate that running emmake make after the correct configure doesn't "just work" but alas.

@tiran
Copy link
MemberAuthor

Two suggestions but otherwise this looks good! It is a bit unfortunate that running emmake make after the correct configure doesn't "just work" but alas.

It does now :)

@tirantiran marked this pull request as ready for review January 12, 2022 09:01
@tirantiran changed the title bpo-40280: Add --with-emscripten-target to build for browser or nodebpo-40280: Add --with-emscripten-target to build for browser or node (GH-30552)Jan 12, 2022
@tirantiran merged commit 43839ba into python:mainJan 12, 2022
@tirantiran deleted the bpo-40280-emscripten-targets branch January 12, 2022 15:08
Sign up for freeto 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.

4 participants

@tiran@emmatyping@the-knights-who-say-ni@bedevere-bot