Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[3.6] bpo-29822: make inspect.isabstract() work during __init_subclass__#1979

Merged
serhiy-storchaka merged 5 commits into
python:3.6python/cpython:3.6from
Soares:backport-fcfe80e-3.6Soares/cpython:backport-fcfe80e-3.6Copy head branch name to clipboard
Jun 7, 2017
Merged

[3.6] bpo-29822: make inspect.isabstract() work during __init_subclass__#1979
serhiy-storchaka merged 5 commits into
python:3.6python/cpython:3.6from
Soares:backport-fcfe80e-3.6Soares/cpython:backport-fcfe80e-3.6Copy head branch name to clipboard

Conversation

@Soares

@Soares Soares commented Jun 7, 2017

Copy link
Copy Markdown
Contributor

This is a backport of #678 to python 3.6. It depends on #527, which was only recently backported (#1282), and I might have made this backport too soon, in which case I apologized.

Also, I'm not entirely sure that I used the cherry picker tool correctly, because (e.g) a new news item snuck into Misc/NEWS when the backport was generated; I'll clean that up shortly, but I recommend having someone double-check that I'm doing this right :-)

Nate added 2 commits June 6, 2017 17:16
…s__. (pythonGH-678)

At the time when an abstract base class' __init_subclass__ runs,
ABCMeta.__new__ has not yet finished running, so in the presence of
__init_subclass__, inspect.isabstract() can no longer depend only on
TPFLAGS_IS_ABSTRACT..
(cherry picked from commit fcfe80e)
Nate and others added 3 commits June 6, 2017 17:21
This PR was generated by cherry_picker.py. There was a merge conflict in Misc/NEWS that I thought I resolved correctly, but the diff of this branch with master shows that the news item is new-to-this-PR. I'm not sure what's going on, but this commit removes the news item. I'd appreciate a second pair of eyes making sure that I'm doing things right.
The cherry-pick apparently introduced a duplicate item in Misc/NEWS, not sure exactly how. The previous commit removed one copy, but I accidentally removed the spurious copy. I *think* this commit will set things right.
@serhiy-storchaka serhiy-storchaka added the type-bug An unexpected behavior, bug, or error label Jun 7, 2017
@serhiy-storchaka serhiy-storchaka merged commit 09b6c0c into python:3.6 Jun 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.