[issue29699] shutil.rmtree should not fail with FileNotFoundError (race condition)

coroa added the comment: Just realised that the "race condition" is triggered consistently on Mac OSX machines working on non-Mac filesystems, where they store extended attributes like last access time in meta files with a `._` prefix. These files are listed by `os.scandir` after their

[issue29699] shutil.rmtree should not fail with FileNotFoundError (race condition)

Irit Katriel added the comment: PR13580 was not declined, the reviewer requested unit tests. If you want to advance this, perhaps you can help fill that in. The other PR looks like a duplicate that should be closed. -- ___ Python tracker

[issue29699] shutil.rmtree should not fail with FileNotFoundError (race condition)

Noam added the comment: Hi Irit, Sorry, I'm still not following, The other issue you stated, states that PR is ready(06-2019), and this was in 2019, is this going to be fixed? or declined (and both should be closed)? What is the current updated status? --

[issue29699] shutil.rmtree should not fail with FileNotFoundError (race condition)

Irit Katriel added the comment: seel also issue37260 -- nosy: +iritkatriel ___ Python tracker ___ ___ Python-bugs-list mailing

[issue29699] shutil.rmtree should not fail with FileNotFoundError (race condition)

Noam added the comment: Is this still alive? If decided to decline PR why is this still open? -- nosy: +noamda ___ Python tracker ___

[issue29699] shutil.rmtree should not fail with FileNotFoundError (race condition)

Jeffrey Kintscher added the comment: I created pull request bpo-29699 to fix this issue. It adds an additional exception handler to ignore FileNotFoundError for most of the try blocks that already handle OSError. I decided not to add it to the initial os.open() call. This should provide the

[issue29699] shutil.rmtree should not fail with FileNotFoundError (race condition)

Change by Jeffrey Kintscher : -- keywords: +patch pull_requests: +13487 stage: -> patch review pull_request: https://github.com/python/cpython/pull/13580 ___ Python tracker

[issue29699] shutil.rmtree should not fail with FileNotFoundError (race condition)

Change by Jeffrey Kintscher : -- nosy: +websurfer5 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue29699] shutil.rmtree should not fail with FileNotFoundError (race condition)

Change by Jesse Farnham : -- nosy: +jesse.farnham ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue29699] shutil.rmtree should not fail with FileNotFoundError (race condition)

Serhiy Storchaka added the comment: Shouldn't this be considered as a new feature? There are no guaranties that shutil.rmtree() should work if files or directories are concurrently removed, or created, or made read-only in other threads or processes. --

[issue29699] shutil.rmtree should not fail with FileNotFoundError (race condition)

Change by STINNER Victor : -- nosy: +serhiy.storchaka ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue29699] shutil.rmtree should not fail with FileNotFoundError (race condition)

Change by Serhiy Storchaka : -- type: crash -> behavior ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue29699] shutil.rmtree should not fail with FileNotFoundError (race condition)

Giampaolo Rodola' added the comment: +1. It looks reasonable to ignore FileNotFoundError on os.rmdir(), os.unlink() and also os.open() and os.scandir(). -- nosy: +giampaolo.rodola ___ Python tracker

[issue29699] shutil.rmtree should not fail with FileNotFoundError (race condition)

New submission from Daniel Kahn Gillmor: There is a race condition in shutil.rmtree, where if a file gets removed between when rmtree plans to remove it and when it gets around to removing it, a FileNotFound exception gets raised. The expected semantics of rmtree imply that if the filesystem

[issue29699] shutil.rmtree should not fail with FileNotFoundError (race condition)

Daniel Kahn Gillmor added the comment: and here is python demonstration script that will build breaker.c and then use it to cause the error to be raised from shutils.rmtree. the output of demo.py looks like this: make: 'breaker' is up to date. Traceback (most recent call last): File