Skip to content

Conversation

@dignissimus
Copy link
Contributor

@dignissimus dignissimus commented Apr 23, 2022

@dignissimus
Copy link
Contributor Author

Would this need to be backported?

@AlexWaygood AlexWaygood added the stdlib Standard Library Python modules in the Lib/ directory label May 4, 2022
Copy link
Member

@iritkatriel iritkatriel left a comment

Choose a reason for hiding this comment

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

LGTM.

@ethanfurman what's your view on back porting this?

@iritkatriel iritkatriel changed the title gh-91078: Return None in TarFile.next when the tarfile is empty gh-91078: Return None from TarFile.next when the tarfile is empty Nov 26, 2022
@ethanfurman ethanfurman added the needs backport to 3.11 only security fixes label Nov 26, 2022
@ethanfurman ethanfurman merged commit 78365b8 into python:main Nov 26, 2022
@miss-islington
Copy link
Contributor

Thanks @dignissimus for the PR, and @ethanfurman for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11.
🐍🍒⛏🤖

@bedevere-bot
Copy link

GH-99798 is a backport of this pull request to the 3.11 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.11 only security fixes label Nov 26, 2022
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 26, 2022
…ty (pythonGH-91850)

(cherry picked from commit 78365b8)

Co-authored-by: Sam Ezeh <[email protected]>
Co-authored-by: Irit Katriel <[email protected]>
@ethanfurman
Copy link
Member

@iritkatriel Backporting sounds good, thanks for the help.

@dignissimus Thanks for the work, sorry it took so long to get merged.

miss-islington added a commit that referenced this pull request Nov 26, 2022
…-91850)

(cherry picked from commit 78365b8)

Co-authored-by: Sam Ezeh <[email protected]>
Co-authored-by: Irit Katriel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stdlib Standard Library Python modules in the Lib/ directory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tarfile.TarFile.next() crashes on empty tar files

6 participants