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

ci: require Python 3.8 for deltachat-rpc-client #4410

Merged
merged 1 commit into from
May 21, 2023

Conversation

link2xt
Copy link
Collaborator

@link2xt link2xt commented May 19, 2023

With Python 3.7 asynchronous tests randomly fail
with "RuntimeError: Event loop is closed" during shutdown. Backtrace of the error includes SafeChildWatcher calls.

Python 3.8 has replaced SafeChildWatcher
with a new ThreadedChildWatcher by default [1]
as a bugfix for
"asyncio.create_subprocess_exec() only works with main event loop" bug [2].

Python 3.7 scheduled end of life is 2023-06-27
according to https://devguide.python.org/versions/.

[1] python/cpython#14344
[2] https://bugs.python.org/issue35621

@link2xt
Copy link
Collaborator Author

link2xt commented May 19, 2023

Related bugreport in pytest-asyncio repository: pytest-dev/pytest-asyncio#548

With Python 3.7 asynchronous tests randomly fail
with "RuntimeError: Event loop is closed" during shutdown.
Backtrace of the error includes `SafeChildWatcher` calls.

Python 3.8 has replaced `SafeChildWatcher`
with a new `ThreadedChildWatcher` by default [1]
as a bugfix for
"asyncio.create_subprocess_exec() only works with main event loop" bug [2].

Python 3.7 scheduled end of life is 2023-06-27
according to <https://devguide.python.org/versions/>.

[1] python/cpython#14344
[2] https://bugs.python.org/issue35621
@link2xt link2xt force-pushed the link2xt/minimum-supported-python-3.8 branch from 2f04b87 to 88ef567 Compare May 20, 2023 20:13
@link2xt link2xt merged commit c68a2e3 into master May 21, 2023
@link2xt link2xt deleted the link2xt/minimum-supported-python-3.8 branch May 21, 2023 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants