Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bpo-37834: Prevent shutil.rmtree exception #15602

Merged
merged 1 commit into from
Aug 29, 2019

Conversation

ned-deily
Copy link
Member

@ned-deily ned-deily commented Aug 29, 2019

when built on non-Windows system without fd system call support,
like older versions of macOS.

https://bugs.python.org/issue37834

when built on non-Windows system without fd system call support,
like older versions of macOS.
@ambv
Copy link
Contributor

ambv commented Aug 29, 2019

Looks fine. Tests pass, manually tested on Windows, too.

@ambv ambv merged commit 7fcc208 into python:master Aug 29, 2019
@miss-islington
Copy link
Contributor

Thanks @ned-deily for the PR, and @ambv for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

@miss-islington
Copy link
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
Contributor

Thanks @ned-deily for the PR, and @ambv for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

Sorry @ned-deily and @ambv, I had trouble checking out the 3.8 backport branch.
Please backport using cherry_picker on command line.
cherry_picker 7fcc2088a50a4ecb80e5644cd195bee209c9f979 3.8

ambv pushed a commit to ambv/cpython that referenced this pull request Aug 29, 2019
when built on non-Windows system without fd system call support,
like older versions of macOS.
(cherry picked from commit 7fcc208)

Co-authored-by: Ned Deily <nad@python.org>
@bedevere-bot
Copy link

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

ambv added a commit that referenced this pull request Aug 29, 2019
when built on non-Windows system without fd system call support,
like older versions of macOS.
(cherry picked from commit 7fcc208)

Co-authored-by: Ned Deily <nad@python.org>
@ned-deily ned-deily deleted the bpo-37834-rmtree branch September 1, 2019 17:54
lisroach pushed a commit to lisroach/cpython that referenced this pull request Sep 10, 2019
when built on non-Windows system without fd system call support,
like older versions of macOS.
DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 2020
when built on non-Windows system without fd system call support,
like older versions of macOS.
websurfer5 pushed a commit to websurfer5/cpython that referenced this pull request Jul 20, 2020
when built on non-Windows system without fd system call support,
like older versions of macOS.
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