From 5a756173369f038c9415af33d25e34ed1dba2a58 Mon Sep 17 00:00:00 2001
From: Pedro Silva <pedro.silva@wirepas.com>
Date: Mon, 13 May 2019 13:16:19 +0300
Subject: [PATCH] Fix docker-build.sh

A few bash variables were protected, causing
the build stage to fail.

Please consider using .ci/build-images instead
---
 container/build_defaults.env |  3 +--
 container/docker-build.sh    | 23 ++++++++++++-----------
 2 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/container/build_defaults.env b/container/build_defaults.env
index 08e8b0d7..d4e5a074 100644
--- a/container/build_defaults.env
+++ b/container/build_defaults.env
@@ -5,7 +5,7 @@ DOCKER_IMAGE_ARM_NAME="wm-gateway-rpi"
 DOCKER_IMAGE_TAG="latest"
 DOCKER_IMAGE_TAG_PYTHON_VERSION="false"
 
-DOCKER_BUILD_CACHE=""
+DOCKER_BUILD_CACHE=" "
 #DOCKER_ARM_BUILD_TARGET="wm-lxgw-rpi"
 #DOCKER_X86_BUILD_TARGET="wm-lxgw"
 
@@ -15,4 +15,3 @@ DOCKER_PLATFORM="x86"
 # use to specify the the messaging or transport version
 #DOCKER_BUILD_ARGS="--build-arg WM_MESSAGING_PKG=wirepas_messaging-1.1.0* \
 #                   --build-arg WM_TRANSPORT_PKG=wirepas_gateway-1.1.0*"
-
diff --git a/container/docker-build.sh b/container/docker-build.sh
index 1114bdee..0b88002a 100755
--- a/container/docker-build.sh
+++ b/container/docker-build.sh
@@ -8,6 +8,7 @@
 #
 
 trap 'echo "Aborting due to errexit on line $LINENO. Exit code: $?" >&2' ERR
+set -x
 set -e
 set -o nounset
 set -o errexit
@@ -44,17 +45,17 @@ _defaults()
 
     # default defaults if not defined in build file
     DOCKER_IMAGE_TAG=${DOCKER_IMAGE_TAG:-"latest"}
-    DOCKER_BUILD_CACHE=${DOCKER_BUILD_CACHE:-""}
+    DOCKER_BUILD_CACHE=${DOCKER_BUILD_CACHE:-" "}
     DOCKER_PLATFORM=${DOCKER_PLATFORM:-"x86"}
-    DOCKER_BUILD_TARGET=${DOCKER_BUILD_TARGET:-""}
+    DOCKER_BUILD_TARGET=${DOCKER_BUILD_TARGET:-" "}
     DOCKER_FILE=${DOCKER_FILE:-"./container/dev/Dockerfile"}
-    DOCKER_REPO=${DOCKER_REPO:-""}
+    DOCKER_REPO=${DOCKER_REPO:-" "}
     DOCKER_PUSH=${DOCKER_PUSH:-"false"}
     DOCKER_SKIP_BUILD=${DOCKER_SKIP_BUILD:-"false"}
-    DOCKER_BUILD_ARGS=${DOCKER_BUILD_ARGS:-""}
+    DOCKER_BUILD_ARGS=${DOCKER_BUILD_ARGS:-" "}
 
-    DOCKER_IMAGE_ARM_NAME=${DOCKER_IMAGE_ARM_NAME:-""}
-    DOCKER_ARM_BUILD_TARGET=${DOCKER_ARM_BUILD_TARGET:-""}
+    DOCKER_IMAGE_ARM_NAME=${DOCKER_IMAGE_ARM_NAME:-" "}
+    DOCKER_ARM_BUILD_TARGET=${DOCKER_ARM_BUILD_TARGET:-" "}
 
     DOCKER_IMAGE_TAG_PYTHON_VERSION=${DOCKER_IMAGE_TAG_PYTHON_VERSION:-"false"}
     DOCKER_BUILD_GIT_HISTORY_LENGTH=${DOCKER_BUILD_GIT_HISTORY_LENGTH:-"10"}
@@ -203,12 +204,13 @@ _build()
 
         _get_build_history || true
         echo "building ${DOCKER_FILE} ${DOCKER_IMAGE_NAME}:${DOCKER_IMAGE_TAG} [${DOCKER_BUILD_CACHE}] ${DOCKER_BUILD_ARGS}"
+        #shellcheck disable=SC2086
         docker build \
-            --compress "${DOCKER_BUILD_CACHE}" \
+            --compress ${DOCKER_BUILD_CACHE} \
            -t "${DOCKER_IMAGE_NAME}":"${DOCKER_IMAGE_TAG}" . \
-           -f "${DOCKER_FILE}" \
-           "${DOCKER_BUILD_TARGET}" \
-           "${DOCKER_BUILD_ARGS}"
+           -f ${DOCKER_FILE} \
+           ${DOCKER_BUILD_TARGET} \
+           ${DOCKER_BUILD_ARGS}
     fi
 }
 
@@ -241,4 +243,3 @@ _main()
 }
 
 _main "${@}"
-