Skip to content

bpo-39492: Fix a reference cycle between reducer_override and a Pickler instance#18266

Merged
miss-islington merged 5 commits into
python:masterfrom
pierreglaser:fix-pickler-reference-cycle
Feb 2, 2020
Merged

bpo-39492: Fix a reference cycle between reducer_override and a Pickler instance#18266
miss-islington merged 5 commits into
python:masterfrom
pierreglaser:fix-pickler-reference-cycle

Conversation

@pierreglaser

@pierreglaser pierreglaser commented Jan 30, 2020

Copy link
Copy Markdown
Contributor

This also needs a backport to 3.8

https://bugs.python.org/issue39492

Automerge-Triggered-By: @pitrou

@pierreglaser

Copy link
Copy Markdown
Contributor Author

@pitrou if you want to take a look.

@pitrou pitrou 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.

Thank you for noticing and for the fix.

Comment thread Modules/_pickle.c
Comment thread Lib/test/pickletester.py Outdated
Comment thread Lib/test/pickletester.py
Comment thread Modules/_pickle.c Outdated
@bedevere-bot

Copy link
Copy Markdown

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

@miss-islington

Copy link
Copy Markdown
Contributor

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

@miss-islington
miss-islington merged commit 0f2f35e into python:master Feb 2, 2020
@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @pierreglaser for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

@pierreglaser
pierreglaser deleted the fix-pickler-reference-cycle branch February 2, 2020 18:57
@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @pierreglaser for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

@miss-islington

Copy link
Copy Markdown
Contributor

I'm having trouble backporting to 3.8. Reason: 'Error 110 while writing to socket. Connection timed out.'. Please retry by removing and re-adding the needs backport to 3.8 label.

@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @pierreglaser for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

@bedevere-bot

Copy link
Copy Markdown

GH-18316 is a backport of this pull request to the 3.8 branch.

pitrou added a commit that referenced this pull request Feb 2, 2020
… Pickler instance (GH-18266) (#18316)

https://bugs.python.org/issue39492

Automerge-Triggered-By: @pitrou
(cherry picked from commit 0f2f35e)

Co-authored-by: Pierre Glaser <pierreglaser@msn.com>
@miss-islington

Copy link
Copy Markdown
Contributor

Sorry, @pierreglaser, I could not cleanly backport this to 3.8 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 0f2f35e15f9fbee44ce042b724348419d8136bc5 3.8

@miss-islington miss-islington self-assigned this Feb 3, 2020
@miss-islington

Copy link
Copy Markdown
Contributor

Sorry @pierreglaser, I had trouble checking out the 3.8 backport branch.
Please backport using cherry_picker on command line.
cherry_picker 0f2f35e15f9fbee44ce042b724348419d8136bc5 3.8

@pierreglaser

Copy link
Copy Markdown
Contributor Author

I'm confused by the bot: was this PR well backported to 3.8? cc @pitrou

@pitrou

pitrou commented Feb 4, 2020

Copy link
Copy Markdown
Member

It was, but I had to do it manually because the bot failed for some reason.

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.

5 participants