8000 GH-120804: Remove `PidfdChildWatcher`, `ThreadedChildWatcher` and `AbstractChildWatcher` from asyncio APIs by kumaraditya303 · Pull Request #120893 · python/cpython · GitHub
[go: up one dir, main page]

Skip to content

GH-120804: Remove PidfdChildWatcher, ThreadedChildWatcher and AbstractChildWatcher from asyncio APIs #120893

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

Merged
merged 2 commits into from
Jun 23, 2024

Conversation

kumaraditya303
Copy link
Contributor
@kumaraditya303 kumaraditya303 commented Jun 23, 2024

@kumaraditya303 kumaraditya303 merged commit 9d2e1ea into python:main Jun 23, 2024
34 checks passed
@kumaraditya303 kumaraditya303 deleted the watcher branch June 23, 2024 13:08
mrahtz pushed a commit to mrahtz/cpython that referenced this pull request Jun 30, 2024
…nd `AbstractChildWatcher` from asyncio APIs (python#120893)
noahbkim pushed a commit to hudson-trading/cpython that referenced this pull request Jul 11, 2024
estyxx pushed a commit to estyxx/cpython that referenced this pull request Jul 17, 2024
…nd `AbstractChildWatcher` from asyncio APIs (python#120893)
@allisonkarlitskaya
Copy link
Contributor
allisonkarlitskaya commented Mar 6, 2025

hi,

Where was it documented that PidfdChildWatcher was deprecated and slated for removal?

https://docs.python.org/3/library/asyncio-policy.html#asyncio.PidfdChildWatcher shows only "added in 3.9". We specifically added a check for this class by name because of the pending deprecation of SafeChildWatcher (which is indeed marked as "Deprecated since version 3.12." on the same page) and now our code is broken with Python 3.14...

See https://bugzilla.redhat.com/show_bug.cgi?id=2350287

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.

2 participants
0