Skip to content

gh-138850: Add --disable-epoll to configure#145768

Open
mtelka wants to merge 5 commits intopython:mainfrom
mtelka:disable-epoll
Open

gh-138850: Add --disable-epoll to configure#145768
mtelka wants to merge 5 commits intopython:mainfrom
mtelka:disable-epoll

Conversation

@mtelka
Copy link
Copy Markdown

@mtelka mtelka commented Mar 10, 2026

@bedevere-app
Copy link
Copy Markdown

bedevere-app bot commented Mar 10, 2026

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@python-cla-bot
Copy link
Copy Markdown

python-cla-bot bot commented Mar 10, 2026

All commit authors signed the Contributor License Agreement.

CLA signed

@encukou
Copy link
Copy Markdown
Member

encukou commented Mar 25, 2026

Looks good! Could you also add docs to https://docs.python.org/3/using/configure.html (maybe a new System Options?) and to https://docs.python.org/3/library/select.html#select.epoll?

Or let me know if you want me to do that.

@mtelka
Copy link
Copy Markdown
Author

mtelka commented Mar 26, 2026

@encukou I'd really appreciate if you'll do that. Thank you.

@vstinner
Copy link
Copy Markdown
Member

@mtelka: Would it make sense to detect illumos in configure and disable epoll support on this OS?

@mtelka
Copy link
Copy Markdown
Author

mtelka commented Mar 27, 2026

@mtelka: Would it make sense to detect illumos in configure and disable epoll support on this OS?

Yes, it would. However I'm not sure it is worth doing.

@vstinner
Copy link
Copy Markdown
Member

I tested the change on Linux:

$ ./configure --disable-epoll && make
$ ./python
>>> import select
>>> select.epoll
Traceback (most recent call last):
  File "<python-input-1>", line 1, in <module>
    select.epoll
AttributeError: module 'select' has no attribute 'epoll'. Did you mean '.poll' instead of '.epoll'?

It works as expected :-)

Copy link
Copy Markdown
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

LGTM.

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.

configure option for epoll disable

3 participants