commit d540d8b0cd9994e5d8b489bab6925c60e92d8087
author Josip Sokcevic <sokcevic@google.com> Wed Oct 12 18:43:49 2022 +0000
committer LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Oct 12 18:43:49 2022 +0000
tree bda936675f0d177dcd96f358695aa56086ab98b1
parent 5c72d1e40c3e0677e49df455627a3dce2113f8e1

[git-cache] Fix remove files recursively

os.remove can't be used to remove non empty directory. Use rmtree from
gclient_utils.

R=jojwang@google.com

Bug: 1372658
Change-Id: I5c08bad9778396cb9b1ec0bebe6287a0f3823fab
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3950469
Reviewed-by: Joanna Wang <jojwang@chromium.org>
Commit-Queue: Joanna Wang <jojwang@chromium.org>
Auto-Submit: Josip Sokcevic <sokcevic@google.com>

git_cache.py

diff --git a/git_cache.py b/git_cache.py
index 4a3c573..1c5a1fe 100755
--- a/git_cache.py
+++ b/git_cache.py
@@ -521,7 +521,7 @@
                                 self.mirror_path.split(os.path.sep)[-1])
     if os.path.exists(recursed_dir):
       self.print('Deleting unexpected directory: %s' % recursed_dir)
-      os.remove(recursed_dir)
+      gclient_utils.rmtree(recursed_dir)
 
     # The folder is <git number>
     gen_number = subprocess.check_output(