[proxy] web.archive.org — ← back | site home | direct (HTTPS) ↗ | proxy home | ◑ dark◐ light
/ cpython Public
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

Jump to bottom

[3.8] bpo-39148: enable ipv6 for datagrams in Proactor (GH-19121) #20169

Merged
merged 1 commit into from May 18, 2020
Merged

[3.8] bpo-39148: enable ipv6 for datagrams in Proactor (GH-19121) #20169

merged 1 commit into from May 18, 2020

Conversation

Copy link
Contributor

miss-islington commented May 18, 2020 •
edited

Ifdef is not necessary, as AF_INET6 is supported from Windows Vista, and other code in overlapped.c uses AF_INET6 and is not ifdef'd.
Change the raised exception so users are not fooled to think it comes from Windows API.

Automerge-Triggered-By: @njsmith
(cherry picked from commit 442634c)

Co-authored-by: Kjell Braden afflux@pentabarf.de

https://bugs.python.org/issue39148

Automerge-Triggered-By: @njsmith

bpo-39148: enable ipv6 for datagrams in Proactor (pythonGH-19121)
bff2305 
Ifdef is not necessary, as AF_INET6 is supported from Windows Vista, and other code in overlapped.c uses AF_INET6 and is not ifdef'd.
Change the raised exception so users are not fooled to think it comes from Windows API.

Automerge-Triggered-By: @njsmith
(cherry picked from commit 442634c)

Co-authored-by: Kjell Braden <afflux@pentabarf.de>
bedevere-bot mentioned this pull request May 18, 2020
Merged
njsmith added the 🤖 automerge PR will be merged once it's been approved and all CI passed label May 18, 2020
Copy link
Contributor Author

miss-islington commented May 18, 2020

@afflux: Status check is done, and it's a success ✅ .

miss-islington merged commit 94d9c5e into python:3.8 May 18, 2020
5 checks passed
miss-islington deleted the backport-442634c-3.8 branch May 18, 2020
Copy link
Contributor Author

miss-islington commented May 18, 2020

@afflux: Status check is done, and it's a success ✅ .

agronholm mannequin mentioned this pull request Apr 10, 2022
Closed
NewUserHa mentioned this pull request May 4, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

njsmith

1st1

asvetlov

Assignees
No one assigned
Labels
🤖 automerge PR will be merged once it's been approved and all CI passed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants