Skip to content

bpo-26660, bpo-35144: Fix permission errors in TemporaryDirectory cleanup. - #10320

Merged
serhiy-storchaka merged 2 commits into
python:masterfrom
serhiy-storchaka:tempfile-cleanup-permissions
May 31, 2019
Merged

bpo-26660, bpo-35144: Fix permission errors in TemporaryDirectory cleanup.#10320
serhiy-storchaka merged 2 commits into
python:masterfrom
serhiy-storchaka:tempfile-cleanup-permissions

Conversation

@serhiy-storchaka

@serhiy-storchakaserhiy-storchaka commented Nov 4, 2018

Copy link
Copy Markdown
Member

TemporaryDirectory.cleanup() failed when non-writeable or non-searchable
files or directories were created inside a temporary directory.

https://bugs.python.org/issue26660

…an up.
TemporaryDirectory.cleanup() failed when non-writeable or non-searchable
files or directories were created inside a temporary directory.
@serhiy-storchakaserhiy-storchaka added the type-bug An unexpected behavior, bug, or error label Nov 4, 2018
techalchemy added a commit to sarugaku/vistir that referenced this pull request Nov 10, 2018
- Backport python/cpython#10320 (not yet merged) for better cleanup on
`shutil.rmtree` failure (using `os.chflags`)
- Add compatibility shims for `PermissionError` and `IsADirectory`
exceptions
- General cleanup
- Fixes#38
Signed-off-by: Dan Ryan <dan@danryan.co>
@serhiy-storchaka
serhiy-storchaka merged commit e9b51c0 into python:masterMay 31, 2019
@serhiy-storchaka
serhiy-storchaka deleted the tempfile-cleanup-permissions branch May 31, 2019 08:30
DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 2020
…anup. (pythonGH-10320)
TemporaryDirectory.cleanup() failed when non-writeable or non-searchable
files or directories were created inside a temporary directory.
@juhuebnerjuhuebner mentioned this pull request Feb 4, 2021
@akaihola

Copy link
Copy Markdown

For anyone still needing to support Python 3.7 and having this issue, you can find a recipe for a work-around in:

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type-bugAn unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@serhiy-storchaka@akaihola@the-knights-who-say-ni@bedevere-bot