From 283bc2c7d0b44dafe5dfca711bcdd5b27ec7e968 Mon Sep 17 00:00:00 2001
From: Yury Semikhatsky <yurys@chromium.org>
Date: Wed, 4 Nov 2020 11:20:47 -0800
Subject: [PATCH] devops: ensure that embedder directory does not exist (#4340)

Otherwise `cp -r from to` will copy content of `from` to a subdirectory in `to`
---
 browser_patches/prepare_checkout.sh | 20 ++++++++++++++++----
 1 file changed, 16 insertions(+), 4 deletions(-)

diff --git a/browser_patches/prepare_checkout.sh b/browser_patches/prepare_checkout.sh
index 6d06370897..8771d3e002 100755
--- a/browser_patches/prepare_checkout.sh
+++ b/browser_patches/prepare_checkout.sh
@@ -124,12 +124,24 @@ git apply --index --whitespace=nowarn $PATCHES_PATH/*
 
 if [[ ! -z "${WEBKIT_EXTRA_FOLDER_PATH}" ]]; then
   echo "-- adding WebKit embedders"
-  cp -r "${WEBKIT_EXTRA_FOLDER_PATH}" ./Tools/Playwright
-  git add Tools/Playwright
+  EMBEDDER_DIR="$PWD/Tools/Playwright"
+  # git status does not show empty directories, check it separately.
+  if [[ -d $EMBEDDER_DIR ]]; then
+    echo "ERROR: $EMBEDDER_DIR already exists! Remove it and re-run the script."
+    exit 1
+  fi
+  cp -r "${WEBKIT_EXTRA_FOLDER_PATH}" $EMBEDDER_DIR
+  git add $EMBEDDER_DIR
 elif [[ ! -z "${FIREFOX_EXTRA_FOLDER_PATH}" ]]; then
   echo "-- adding juggler"
-  cp -r "${FIREFOX_EXTRA_FOLDER_PATH}" ./juggler
-  git add juggler
+  EMBEDDER_DIR="$PWD/juggler"
+  # git status does not show empty directories, check it separately.
+  if [[ -d $EMBEDDER_DIR ]]; then
+    echo "ERROR: $EMBEDDER_DIR already exists! Remove it and re-run the script."
+    exit 1
+  fi
+  cp -r "${FIREFOX_EXTRA_FOLDER_PATH}" $EMBEDDER_DIR
+  git add $EMBEDDER_DIR
 fi
 
 git commit -a --author="playwright-devops <devops@playwright.dev>" -m "chore: bootstrap build #$BUILD_NUMBER"