FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add cleanup_socket param on create_unix_server() by fantix · Pull Request #623 · MagicStack/uvloop · GitHub

Repository navigation

Add cleanup_socket param on create_unix_server() - #623

Closed
fantix wants to merge 1 commit into
masterfrom
cleanup_socket
Closed

fantix wants to merge 1 commit into
masterfrom
cleanup_socket

Conversation

fantix commented Aug 28, 2024 •
edited
Loading

Copy link
Copy Markdown
Member

This is derived from python/cpython#111483 but available on all Python versions with uvloop, only that it's only enabled by default for Python 3.13 and above to be consistent with CPython behavior.

Sample CI run on 3.13

This is derived from python/cpython#111483 but available on
all Python versions with uvloop, only that it's only enabled
by default for Python 3.13 and above to be consistent with
CPython behavior.
fantix requested a review from 1st1 August 28, 2024 18:09
fantix mentioned this pull request Aug 28, 2024
fantix added a commit that referenced this pull request Sep 2, 2024
This is derived from python/cpython#111483 but available on
all Python versions with uvloop, only that it's only enabled
by default for Python 3.13 and above to be consistent with
CPython behavior.

* Also add Python 3.13 to CI (#610)
* Enable CI in debug mode
* Fix CI to include [dev]

---------

Co-authored-by: Fantix King <fantix.king@gmail.com>

fantix commented Sep 2, 2024

Copy link
Copy Markdown
Member Author

Merged in #610, @1st1 feel free to drop post-merge review comments.

fantix closed this Sep 2, 2024
fantix deleted the cleanup_socket branch September 2, 2024 20:58
fantix added a commit that referenced this pull request Oct 14, 2024
Changes
=======

* Add cleanup_socket param on create_unix_server() (#623)
  (by @fantix in d6114d2)

Fixes
=====

* Use cythonized SO_REUSEPORT rather than the unwrapped native one. (#609)
  (by @ptribble in 4083a94 for #550)

* UDP errors should result in protocol.error_received (#601)
  (by @jensbjorgensen in 3c3bbef)

* Updates for Cython3 (#587)
  (by @alan-brooks in 3fba9fa for #587)

* Test with Python 3.13 (#610)
  (by @edgarrmondragon in fb5a139)
fantix mentioned this pull request Oct 14, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

1 participant


Back | FazBrowse Home | New Git URL