From f2c04ff675719b863b443f0a2d5b9d320f767a01 Mon Sep 17 00:00:00 2001
From: Yo Chiang <yochiang@google.com>
Date: Thu, 27 Feb 2020 16:38:39 +0800
Subject: [PATCH] gsi-pubsign-release.sh print verbose log

Bug: 149803895
Test: ./gsi-pubsign-release.sh aosp_arm64 aosp_arm64_pubsign \
        aosp_arm64-artifact_path aosp_arm64_pubsign-artifact_path
Merged-In: I71ca710e3d2d7305112e118e8d19c3b07d6f028c
Change-Id: I71ca710e3d2d7305112e118e8d19c3b07d6f028c
(cherry picked from commit fe9bb151c0144f2c988032814a913912fdc3d551)
---
 gsi/gsi-pubsign-release.sh | 30 ++++++++++++++++++++++--------
 1 file changed, 22 insertions(+), 8 deletions(-)

diff --git a/gsi/gsi-pubsign-release.sh b/gsi/gsi-pubsign-release.sh
index 653f0f6c4..9cd4232a7 100755
--- a/gsi/gsi-pubsign-release.sh
+++ b/gsi/gsi-pubsign-release.sh
@@ -1,6 +1,6 @@
 #!/bin/bash
 
-set -e
+set -ex
 
 function usage {
   cat <<EOF
@@ -47,17 +47,31 @@ if [[ -z "${DIST_DIR}" ]]; then
 fi
 
 # Create output directory if not already present
+mkdir -p "${DIST_DIR}" || true
+
 if [[ ! -d "${DIST_DIR}" ]]; then
-  mkdir -p "${DIST_DIR}"
+  echo >&2 'Cannot create $DIST_DIR or $DIST_DIR is non-existence'
+  exit 1
 fi
 
+# Show the artifacts to be copied in the log
+echo "Artifacts to copy:"
+find "${SRC_DIR}" || true
+echo
+
 # Don't copy logs/ and files whose name starts with $SRC_PRODUCT
 rsync --verbose --archive --copy-links --exclude='logs' \
-  --exclude="${SRC_PRODUCT}-*" "${SRC_DIR}/" "${DIST_DIR}"
+  --exclude='*.zip' "${SRC_DIR}/" "${DIST_DIR}"
 
-# Rename ${SRC_PRODUCT}-xxx.yyy to ${TARGET_PRODUCT}-xxx.yyy
-for src_path in $(find "${SRC_DIR}" -type f -name "${SRC_PRODUCT}-*") ; do
-  src_file="$(basename ${src_path})"
-  target_file="${src_file/${SRC_PRODUCT}/${TARGET_PRODUCT}}"
-  cp -v "${src_path}" "${DIST_DIR}/${target_file}"
+# Rename ${SRC_PRODUCT}-xxx.zip to ${TARGET_PRODUCT}-xxx.zip
+ZIP_PATHNAMES="$(find "${SRC_DIR}" -type f -name '*.zip')"
+
+echo "ZIP files to be copied and renamed:"
+echo "${ZIP_PATHNAMES}"
+echo
+
+for SRC_PATHNAME in ${ZIP_PATHNAMES} ; do
+  SRC_FILENAME="$(basename ${SRC_PATHNAME})"
+  TARGET_FILENAME="${SRC_FILENAME/${SRC_PRODUCT}/${TARGET_PRODUCT}}"
+  cp -v "${SRC_PATHNAME}" "${DIST_DIR}/${TARGET_FILENAME}"
 done