Skip to content

[3.10] bpo-45083: Include the exception class qualname when formatting an exception (GH-28119)#28134

Merged
ambv merged 2 commits into
python:3.10from
miss-islington:backport-b4b6342-3.10
Sep 8, 2021
Merged

[3.10] bpo-45083: Include the exception class qualname when formatting an exception (GH-28119)#28134
ambv merged 2 commits into
python:3.10from
miss-islington:backport-b4b6342-3.10

Conversation

@miss-islington

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

Copy link
Copy Markdown
Contributor

Co-authored-by: Erlend Egeberg Aasland erlend.aasland@innova.no
(cherry picked from commit b4b6342)

Co-authored-by: Irit Katriel 1055913+iritkatriel@users.noreply.github.com

https://bugs.python.org/issue45083

…ception (pythonGH-28119)

Co-authored-by: Erlend Egeberg Aasland <erlend.aasland@innova.no>
(cherry picked from commit b4b6342)

Co-authored-by: Irit Katriel <1055913+iritkatriel@users.noreply.github.com>
@miss-islington

Copy link
Copy Markdown
Contributor Author

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

@miss-islington

Copy link
Copy Markdown
Contributor Author

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

1 similar comment
@miss-islington

Copy link
Copy Markdown
Contributor Author

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

@miss-islington

Copy link
Copy Markdown
Contributor Author

@iritkatriel and @ambv: Status check is done, and it's a failure ❌ .

@ambv

ambv commented Sep 3, 2021

Copy link
Copy Markdown
Contributor

Build failures are because PyType_GetQualName doesn't exist in Python 3.10.

Comment thread Include/internal/pycore_object.h
@ambv ambv requested a review from pablogsal September 3, 2021 16:53
@ambv

ambv commented Sep 3, 2021

Copy link
Copy Markdown
Contributor

@pablogsal, without this change C-based exceptions from inner classes have invalid names when printed.

@pablogsal

Copy link
Copy Markdown
Member

Let's merge this on 3.10.1.

This is a lot of C code for a release candidate and is not technically a bug

@ambv ambv self-assigned this Sep 3, 2021
@ambv ambv removed the DO-NOT-MERGE label Sep 8, 2021
@ambv ambv merged commit 6b996d6 into python:3.10 Sep 8, 2021
@miss-islington miss-islington deleted the backport-b4b6342-3.10 branch September 8, 2021 16:32
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