Skip to content

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

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
Merged
tiran merged 2 commits into from
Jan 9, 2022
Merged

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

tiran merged 2 commits into from
Jan 9, 2022

Conversation

@emmatyping
Copy link
Member

@emmatypingemmatyping commented Jan 9, 2022•
edited by bedevere-bot
Loading

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

@emmatyping
bpo-40280: Fix Emscripten build config.site
9d4553b
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•
edited
Loading

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

erlend-aasland
erlend-aasland approved these changes Jan 9, 2022
View reviewed changes
Copy link
Contributor

@erlend-aaslanderlend-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
MemberAuthor

emmatyping 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 commented Jan 9, 2022

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

@emmatypingemmatyping changed the title bpo-40280: Update Emscripten build config.sitebpo-40280: Disable epoll_create in Emscripten config.siteJan 9, 2022
ngnpope
ngnpope suggested changes Jan 9, 2022
View reviewed changes
@emmatyping
Remove out of date line from config.site
0e96b83
Co-authored-by: nick.pope@infogrid.io
@emmatypingemmatyping mentioned this pull request Jan 9, 2022
Closed
@tirantiran changed the title bpo-40280: Disable epoll_create in Emscripten config.sitebpo-40280: Disable epoll_create in Emscripten config.site (GH-30494)Jan 9, 2022
@tirantiran merged commit 5c66414 into python:mainJan 9, 2022
@SimonBiggsSimonBiggsmannequin mentioned this pull request Jun 7, 2022
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tirantirantiran approved these changes

@erlend-aaslanderlend-aaslanderlend-aasland approved these changes

+1 more reviewer

@ngnpopengnpopengnpope requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@emmatyping@sobolevn@corona10@erlend-aasland@tiran@ngnpope@the-knights-who-say-ni@bedevere-bot