Skip to content
This repository has been archived by the owner on Sep 13, 2024. It is now read-only.

Fix compatibility with Python 3.13 #176

Merged
merged 2 commits into from
Aug 17, 2024
Merged

Conversation

marmarek
Copy link
Contributor

@marmarek marmarek commented Aug 16, 2024

_UnixSelectorEventLoop in Python 3.13 expect _unix_server_sockets dict, add one. It's later used (among others) in _stop_serving(), but not in close() directly, so just adding attribute is enough.

The related CPython change: 74b868f636a "gh-111246: Remove listening Unix socket on close (#111483)"

PR Checklist:

  • All new features have been tested
  • All new features have been documented
  • I have read the CONTRIBUTING.md file
  • I will abide by the code of conduct

_UnixSelectorEventLoop in Python 3.13 expect _unix_server_sockets dict,
add one. It's later used (among others) in _stop_serving(), but not in
close() directly, so just adding attribute is enough.

The related CPython change: 74b868f636a "gh-111246: Remove listening Unix socket on close (#111483)"

Add a test for unix sockets too.
@marmarek marmarek marked this pull request as ready for review August 16, 2024 21:20
Copy link
Member

@freakboy3742 freakboy3742 left a comment

Choose a reason for hiding this comment

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

Thanks for the patch. I've made a couple of minor corrections, but otherwise this looks good.

Just so you're aware: we will likely be archiving this project in the near future. PyGObject recently merged a patch that added native asyncio support, which removes the need for Gbulb as an external project. See #32 for details.

tests/test_glib_events.py Outdated Show resolved Hide resolved
tests/test_glib_events.py Outdated Show resolved Hide resolved
changes/176.bugfix.rst Outdated Show resolved Hide resolved
@freakboy3742 freakboy3742 merged commit 0f9781b into beeware:main Aug 17, 2024
9 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants