Skip to content

[3.10] bpo-45121: Fix RecursionError when calling Protocol.__init__ from a subclass' __init__ (GH-28206)#28232

Merged
ambv merged 1 commit into
python:3.10from
miss-islington:backport-c11956a-3.10
Sep 8, 2021
Merged

[3.10] bpo-45121: Fix RecursionError when calling Protocol.__init__ from a subclass' __init__ (GH-28206)#28232
ambv merged 1 commit into
python:3.10from
miss-islington:backport-c11956a-3.10

Conversation

@miss-islington

@miss-islington miss-islington commented Sep 8, 2021

Copy link
Copy Markdown
Contributor

(cherry picked from commit c11956a)

Co-authored-by: Yurii Karabas 1998uriyyo@gmail.com

https://bugs.python.org/issue45121

…ubclass' __init__ (pythonGH-28206)

(cherry picked from commit c11956a)

Co-authored-by: Yurii Karabas <1998uriyyo@gmail.com>
@miss-islington

Copy link
Copy Markdown
Contributor Author

@uriyyo and @Fidget-Spinner: Status check is done, and it's a success ✅ .

@miss-islington

Copy link
Copy Markdown
Contributor Author

@uriyyo and @Fidget-Spinner: Status check is done, and it's a success ✅ .

1 similar comment
@miss-islington

Copy link
Copy Markdown
Contributor Author

@uriyyo and @Fidget-Spinner: Status check is done, and it's a success ✅ .

@Fidget-Spinner

Fidget-Spinner commented Sep 8, 2021

Copy link
Copy Markdown
Member

@pablogsal I'm requesting to cherry pick this into 3.10.0 please.

As described by Shantanu in the bpo https://bugs.python.org/issue45121, a fix for a previous typing bug caused RecursionError, which would make his use case crash.

CC: @ambv too, thanks.

@ambv ambv merged commit c081649 into python:3.10 Sep 8, 2021
@miss-islington miss-islington deleted the backport-c11956a-3.10 branch September 8, 2021 15:05
pablogsal pushed a commit that referenced this pull request Sep 19, 2021
…ubclass' __init__ (GH-28206) (GH-28232)

(cherry picked from commit c11956a)

Co-authored-by: Yurii Karabas <1998uriyyo@gmail.com>
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.

6 participants