Handle unbound arrays correctly

The change cfe54b794b3814f288009b796e82668ba8846ca1 missed
handling unbound arrays with `set -u`.

Issue: RELENG-2013
Change-Id: Ic37ce0404f13bff04aa4122c9b1c91add78e2145
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

diff --git a/releasenotes/notes/handle-unbound-arrays-9a31a444687d18ce.yaml b/releasenotes/notes/handle-unbound-arrays-9a31a444687d18ce.yaml
new file mode 100644 (file)
index 0000000..1be44d8
--- /dev/null
+++ b/releasenotes/notes/handle-unbound-arrays-9a31a444687d18ce.yaml
@@ -0,0 +1,14 @@
+---
+fixes:
+  - |
+    Fix error with handling unbound arrays for search extensions, when using
+    `set -u`. The correct way of using this.
+
+    .. code-block:: bash
+
+       set -u
+       arr=()
+       echo "output: '${arr[@]}'"
+       bash: arr[@]: unbound variable
+       echo "output: '${arr[@]:-}'"
+       foo: ''

diff --git a/shell/logs-deploy.sh b/shell/logs-deploy.sh
index a00f0a7..23660d7 100644 (file)
--- a/shell/logs-deploy.sh
+++ b/shell/logs-deploy.sh
@@ -21,9 +21,10 @@ else
 
     # Handle multiple search extensions as separate values to '-p|--pattern'
     set -f # Disable pathname expansion
+    search_exts=()
     IFS=' ' read -r -a search_exts <<< "${ARCHIVE_ARTIFACTS:-}"
     pattern_opts=()
-    for search_ext in "${search_exts[@]}";
+    for search_ext in "${search_exts[@]:-}";
     do
         pattern_opts+=("-p" "$search_ext")
     done