[PATCH] D79802: Fixed a problem removing temp files

[Julian Lettner via Phabricator via llvm-commits]

yln added a comment.

Hi, thanks for tackling this!

I am not on Windows, so I can't reproduce this.  Would you be able to debug this so we understand what the problem is? For example, printing the exception object or looking into the error handler of `shutil.rmtree(... , onerror=..)` might shed light on things [1].

The docs say that `os.rmdir()` raises when the directory isn't empty, so maybe it's not the best tool here.

Does the following work?

  # Sleep and retry
  time.sleep(2) # what would be a good value here?
  shutil.rmtree(..., ignore_errors=True)

Thanks!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79802/new/

https://reviews.llvm.org/D79802