From 451b9513e4d224d47e3d4d2d543e1bc834b78794 Mon Sep 17 00:00:00 2001
From: Scott Little <scott.little@windriver.com>
Date: Wed, 2 Dec 2020 12:08:27 -0500
Subject: [PATCH] Fix selection of release specific mock prototype

Commit https://review.opendev.org/c/starlingx/root/+/762700
was intended to include the code to select the release specific
mock config prototype.

    e.g. mock.cfg.centos7.proto

As delivered, it would always select the default.

    e.g. mock.cfg.proto

Other improvements:
- remove some trailing whitespace
- improved cleanup of temorary directories

Closes-Bug: 1906547
Signed-off-by: Scott Little <scott.little@windriver.com>
Change-Id: I3e988fb0e861efcf9cd16b8a46b74398dbb1db17
---
 centos-mirror-tools/download_mirror.sh |  9 +++++
 toCOPY/generate-centos-repo.sh         | 47 +++++++++++++++++++-------
 toCOPY/populate_downloads.sh           | 12 +++++--
 3 files changed, 54 insertions(+), 14 deletions(-)

diff --git a/centos-mirror-tools/download_mirror.sh b/centos-mirror-tools/download_mirror.sh
index e0deb3fb..c7adfbc8 100755
--- a/centos-mirror-tools/download_mirror.sh
+++ b/centos-mirror-tools/download_mirror.sh
@@ -10,6 +10,15 @@ source $DOWNLOAD_MIRROR_DIR/../toCOPY/lst_utils.sh
 export DL_MIRROR_LOG_DIR="${DL_MIRROR_LOG_DIR:-./logs}"
 export DL_MIRROR_OUTPUT_DIR="${DL_MIRROR_OUTPUT_DIR:-./output/stx/CentOS}"
 
+cleanup () {
+    if [ -e "${TMP_LST_DIR}" ]; then
+        \rm -rf ${TMP_LST_DIR}
+    fi
+}
+
+trap "cleanup ; exit 1" INT HUP TERM QUIT
+trap "cleanup" EXIT
+
 # A temporary compatability step to save download time
 # during the shift to the new DL_MIRROR_OUTPUT_DIR location.
 #
diff --git a/toCOPY/generate-centos-repo.sh b/toCOPY/generate-centos-repo.sh
index deedaa75..48eadcf9 100755
--- a/toCOPY/generate-centos-repo.sh
+++ b/toCOPY/generate-centos-repo.sh
@@ -67,7 +67,8 @@ cleanup () {
     fi
 }
 
-trap "cleanup ; exit 1" INT
+trap "cleanup ; exit 1" INT HUP TERM QUIT
+trap "cleanup" EXIT
 
 if [ -z "$MY_REPO" ]; then
     echo "\$MY_REPO is not set. Ensure you are running this script"
@@ -128,11 +129,16 @@ timestamp="$(date +%F_%H%M)"
 mock_cfg_prefix="mock.cfg"
 mock_cfg_default_suffix="proto"
 mock_cfg_suffix="${mock_cfg_default_suffix}"
-if [ -f /etc/os-release ]; then
-    mock_cfg_distro="$(source /etc/os-release; echo ${ID}${VERSION_ID}.proto)"
-fi
+mock_cfg_distro=""
+mock_cfg_release_prefix=${mock_cfg_prefix}
 mock_cfg_dir=$MY_REPO/build-tools/repo_files
 mock_cfg_dest_dir=$MY_REPO/centos-repo
+if [ -f /etc/os-release ]; then
+    mock_cfg_distro="$(source /etc/os-release; echo ${ID}${VERSION_ID})"
+    if [ ! -z "${mock_cfg_distro}" ]; then
+        mock_cfg_release_prefix=${mock_cfg_prefix}.${mock_cfg_distro}
+    fi
+fi
 comps_xml_file=$MY_REPO/build-tools/repo_files/comps.xml
 comps_xml_dest_dir=$MY_REPO/centos-repo/Binary
 
@@ -412,7 +418,7 @@ copy_with_backup () {
 
     if [ ! -d ${dest_dir} ]; then
         dest_file="$2"
-        dest_dir=$(dir_name ${dest_file})
+        dest_dir=$(dirname ${dest_file})
         if [ ! -d ${dest_dir} ]; then
             echo "destination directory '${dest_dir}' does not exist!"
             exit 1
@@ -467,15 +473,32 @@ done
 
 echo "Copying mock.cfg.proto file."
 
-# First look for layer specific file to copy.
-mock_cfg_file="${mock_cfg_dir}/${mock_cfg_prefix}.${layer}.${mock_cfg_suffix}"
-if [ -f "$mock_cfg_file" ]; then
-    copy_with_backup ${mock_cfg_file} ${mock_cfg_dest_dir}/${mock_cfg_prefix}.${layer}.${mock_cfg_default_suffix}
+#
+# There are several mock.cfg.proto to choose from.
+# They may be specific to release (e.g. centos7/8),
+# specific to layer (e.g. distro), or both.
+#
+
+# First look for release specific, layer specific file to copy.
+mock_cfg_file="${mock_cfg_dir}/${mock_cfg_release_prefix}.${layer}.${mock_cfg_suffix}"
+if [ ! -f "${mock_cfg_file}" ]; then
+    # Substitute release default, layer specific file to copy.
+    mock_cfg_file="${mock_cfg_dir}/${mock_cfg_prefix}.${layer}.${mock_cfg_suffix}"
+fi
+if [ -f "${mock_cfg_file}" ]; then
+    echo "copy_with_backup '${mock_cfg_file}' '${mock_cfg_dest_dir}/${mock_cfg_prefix}.${layer}.${mock_cfg_default_suffix}'"
+    copy_with_backup "${mock_cfg_file}" "${mock_cfg_dest_dir}/${mock_cfg_prefix}.${layer}.${mock_cfg_default_suffix}"
 fi
 
-# Always copy the default
-mock_cfg_file=${mock_cfg_dir}/${mock_cfg_prefix}.${mock_cfg_suffix}
-copy_with_backup ${mock_cfg_file} ${mock_cfg_dest_dir}/${mock_cfg_prefix}.${mock_cfg_default_suffix}
+# Always copy the default (with respect to layer)
+# First look for release specific, layer default file to copy.
+mock_cfg_file="${mock_cfg_dir}/${mock_cfg_release_prefix}.${mock_cfg_suffix}"
+if [ ! -f "${mock_cfg_file}" ]; then
+    # Substitute release default, layer default file to copy.
+    mock_cfg_file="${mock_cfg_dir}/${mock_cfg_prefix}.${mock_cfg_suffix}"
+fi
+echo "copy_with_backup '${mock_cfg_file}' '${mock_cfg_dest_dir}/${mock_cfg_prefix}.${mock_cfg_default_suffix}'"
+copy_with_backup "${mock_cfg_file}" "${mock_cfg_dest_dir}/${mock_cfg_prefix}.${mock_cfg_default_suffix}"
 
 
 echo "Copying contents from other list files."
diff --git a/toCOPY/populate_downloads.sh b/toCOPY/populate_downloads.sh
index 7d8f7b44..e434becf 100755
--- a/toCOPY/populate_downloads.sh
+++ b/toCOPY/populate_downloads.sh
@@ -20,6 +20,15 @@ usage () {
     echo "  --mirror-dir=<dir>: Set the mirror directory.  This is where the previously download tarballs are located."
 }
 
+cleanup () {
+    if [ -e "${TMP_LST_DIR}" ]; then
+        \rm -rf ${TMP_LST_DIR}
+    fi
+}
+
+trap "cleanup ; exit 1" INT HUP TERM QUIT
+trap "cleanup" EXIT
+
 mirror_dir=""
 
 if [ -z "$MY_REPO" ]; then
@@ -68,6 +77,7 @@ extra_downloads_template="extra_downloads.lst"
 
 TMP_LST_DIR=$(mktemp -d /tmp/tmp_lst_dir_XXXXXX)
 mkdir -p $TMP_LST_DIR
+
 tarball_lst="$TMP_LST_DIR/${tarball_downloads_template}"
 extra_downloads_lst="$TMP_LST_DIR/${extra_downloads_template}"
 merge_lst ${config_dir} ${distro} ${tarball_downloads_template} > ${tarball_lst}
@@ -111,5 +121,3 @@ done
 for x in ${extra_downloads}; do
     ln -sf ${mirror_dir}/downloads/$x ${downloads_dir}
 done
-
-\rm -rf ${TMP_LST_DIR}