Skip to content

Conversation

@emmatyping
Copy link
Member

@emmatyping emmatyping commented Jan 9, 2022

With the fix in 994f90c
building for Emscripten started failing with epoll link
errors as can be seen here:
https://github.com/ethanhs/python-wasm/runs/4745952720

I believe this shouldn't require a news entry.

https://bugs.python.org/issue40280

With the fix in 994f90c
building for Emscripten started failing with epoll link
errors as can be seen here:
https://github.com/ethanhs/python-wasm/runs/4745952720
@sobolevn
Copy link
Member

sobolevn commented Jan 9, 2022

@ethanhs should it be back-ported? 🤔

CC @corona10

@corona10
Copy link
Member

corona10 commented Jan 9, 2022

@sobolevn
Supporting WASM is a new feature from Python 3.11
I don't think that the backport is needed.

Copy link
Contributor

@erlend-aasland erlend-aasland left a comment

Choose a reason for hiding this comment

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

lgtm

@emmatyping
Copy link
Member Author

Yes, since this is new in 3.11 there should be no need for any backporting.

@erlend-aasland
Copy link
Contributor

I'd update the PR title to more accurately reflect the actual change, though.

@emmatyping emmatyping changed the title bpo-40280: Update Emscripten build config.site bpo-40280: Disable epoll_create in Emscripten config.site Jan 9, 2022
Co-authored-by: nick.pope@infogrid.io
@tiran tiran changed the title bpo-40280: Disable epoll_create in Emscripten config.site bpo-40280: Disable epoll_create in Emscripten config.site (GH-30494) Jan 9, 2022
@tiran tiran merged commit 5c66414 into python:main Jan 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants