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

bpo-40280: Disable epoll_create in Emscripten config.site (GH-30494) #30494

Merged
merged 2 commits into from Jan 9, 2022

Conversation

@ethanhs
Copy link
Contributor

@ethanhs ethanhs 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 sobolevn commented Jan 9, 2022

@ethanhs should it be back-ported? 🤔

CC @corona10

@corona10
Copy link
Member

@corona10 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

lgtm

@ethanhs
Copy link
Contributor Author

@ethanhs ethanhs commented Jan 9, 2022

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

@erlend-aasland
Copy link
Contributor

@erlend-aasland erlend-aasland commented Jan 9, 2022

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

@ethanhs ethanhs changed the title bpo-40280: Update Emscripten build config.site bpo-40280: Disable epoll_create in Emscripten config.site Jan 9, 2022
Tools/wasm/config.site-wasm32-emscripten Outdated Show resolved Hide resolved
Co-authored-by: nick.pope@infogrid.io
tiran
tiran approved these changes Jan 9, 2022
@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
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
8 participants