Skip to content

bpo-40280: Add --with-emscripten-target to build for browser or node (GH-30552) - #30552

Merged
tiran merged 9 commits into
python:mainfrom
tiran:bpo-40280-emscripten-targets
Jan 12, 2022
Merged

bpo-40280: Add --with-emscripten-target to build for browser or node (GH-30552)#30552
tiran merged 9 commits into
python:mainfrom
tiran:bpo-40280-emscripten-targets

Conversation

@tiran

@tirantiran commented Jan 11, 2022

Copy link
Copy Markdown
Member

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>
@tiran
tiranforce-pushed the bpo-40280-emscripten-targets branch from a1aab67 to b5b4504CompareJanuary 11, 2022 23:46
Comment threadconfigure Outdated
@brettcannon
brettcannon self-requested a review January 12, 2022 01:08
Comment threadconfigure.ac

@emmatypingemmatyping left a comment

Copy link
Copy Markdown
Member

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
Copy Markdown
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 :)

@tiran
tiran 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
@tiran
tiran merged commit 43839ba into python:mainJan 12, 2022
@tiran
tiran 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