Skip to content

bpo-36818: Add PyInterpreterState.runtime field. - #13129

Merged
ericsnowcurrently merged 4 commits into
python:masterfrom
ericsnowcurrently:interp-runtime-field
Jun 1, 2019
Merged

bpo-36818: Add PyInterpreterState.runtime field.#13129
ericsnowcurrently merged 4 commits into
python:masterfrom
ericsnowcurrently:interp-runtime-field

Conversation

@ericsnowcurrently

@ericsnowcurrentlyericsnowcurrently commented May 6, 2019•
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

@ericsnowcurrently
ericsnowcurrentlyforce-pushed the interp-runtime-field branch 2 times, most recently from 33fc33e to 7a22095CompareMay 7, 2019 13:30
vstinner
vstinner requested changes May 10, 2019
View reviewed changes

@vstinnervstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I disagree with the overall approach. Maybe I misunderstood you.
https://bugs.python.org/issue36818#msg342043

Comment threadInclude/internal/pycore_object.h Outdated
Comment threadInclude/internal/pycore_pystate.h Outdated
@bedevere-bot

bedevere-bot commented May 10, 2019

Copy link
Copy Markdown

When you're done making the requested changes, leave the comment: I have made the requested changes; please review again.

And if you don't make the requested changes, you will be poked with soft cushions!

@vstinner

vstinner commented May 10, 2019

Copy link
Copy Markdown
Member

Eric is working on a different approach: https://bugs.python.org/issue36854

@ericsnowcurrently

ericsnowcurrently commented May 31, 2019

Copy link
Copy Markdown
MemberAuthor

The GC stuff is orthogonal to adding PyInterpreterState.runtime. I still plan on merging this after resolving merge conflicts and any feedback.

@ericsnowcurrently

ericsnowcurrently commented Jun 1, 2019•
edited
Loading

Copy link
Copy Markdown
MemberAuthor

FYI, I've trimmed the PR down. The changes I've removed can wait until later.

@ericsnowcurrently
ericsnowcurrently merged commit 396e0a8 into python:masterJun 1, 2019
@ericsnowcurrently
ericsnowcurrently deleted the interp-runtime-field branch June 1, 2019 03:16
vstinner added a commit that referenced this pull request Jun 4, 2019
@vstinner
Revert "bpo-36818: Add PyInterpreterState.runtime field. (gh-13129)" (G…
0fd2c30
…H-13795)
This reverts commit 396e0a8.
DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 2020
@ericsnowcurrently@DinoV
bpo-36818: Add PyInterpreterState.runtime field. (pythongh-13129)
f79dabd
https://bugs.python.org/issue36818
DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 2020
@vstinner@DinoV
Revert "bpo-36818: Add PyInterpreterState.runtime field. (pythongh-13129
72ba7cf
)" (pythonGH-13795)
This reverts commit 396e0a8.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vstinnervstinnervstinner requested changes
@brettcannonbrettcannonAwaiting requested review from brettcannon
@encukouencukouAwaiting requested review from encukou
@ncoghlanncoghlanAwaiting requested review from ncoghlan
@warsawwarsawAwaiting requested review from warsaw

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ericsnowcurrently@bedevere-bot@vstinner@the-knights-who-say-ni