commit 718f0bd0d00299f4b2da2afa9a91e4c1f8798c3c
author Ravi Mistry <rmistry@google.com> Thu Jan 13 16:24:23 2022 -0500
committer SkCQ <skcq-be@skia-corp.google.com.iam.gserviceaccount.com> Thu Jan 13 21:39:20 2022 +0000
tree d8ade0d1b3074175d52b049bcd13b13b3ccae983
parent 93fc06bfd0cecf7cca0d360b2a6e5c208150df7e

[fiddle] Remove duplicate skia checkout from fiddler's release

This caused the build to slow down.

Bug: skia:12816
Change-Id: Ic954977a000de38a6877bfb36f7dd42dae4e370e
Reviewed-on: https://skia-review.googlesource.com/c/buildbot/+/494217
Reviewed-by: Leandro Lovisolo <lovisolo@google.com>
Commit-Queue: Ravi Mistry <rmistry@google.com>

fiddlek/BUILD.bazel

diff --git a/fiddlek/BUILD.bazel b/fiddlek/BUILD.bazel
index b8df68e..01a0858 100644
--- a/fiddlek/BUILD.bazel
+++ b/fiddlek/BUILD.bazel
@@ -103,9 +103,12 @@
     run_commands_skia = [
         # Builds will use libs from SwiftShader.
         "git clone https://swiftshader.googlesource.com/SwiftShader /tmp/swiftshader",
-        # Move the skia checkout into the dir used by the fiddler k8s config.
+        # Copy the skia checkout into the dir used by the fiddler k8s config.
+        # We cannot do a move because we get a "Permission denied".
         "mkdir /tmp/skia",
         "cp -r /usr/local/share/fiddler/skia /tmp/skia/",
+        # Cleanup the old skia checkout location in /usr/local/share/fiddler/skia
+        "rm -rf /usr/local/share/fiddler/skia/*",
         # Build again so that first runs are fast in the new container.
         "/tmp/depot_tools/ninja -C /tmp/skia/skia/out/Static",
     ],