Browse Source

Silence shellcheck warnings (#279)

* SC1091: the `config` file might not be present, which is normal.
* SC2086: Double quote to prevent globbing and word splitting.

Tested clean output using: `find -name "*.sh" -exec shellcheck -x {} \;`
pull/289/head
Hugo Hromic 3 years ago
committed by XECDesign
parent
commit
407663a57f
2 changed files with 3 additions and 2 deletions
  1. +1
    -0
      build-docker.sh
  2. +2
    -2
      build.sh

+ 1
- 0
build-docker.sh View File

@ -15,6 +15,7 @@ fi
set -e
if [ -f config ]; then
# shellcheck disable=SC1091
source config
fi

+ 2
- 2
build.sh View File

@ -203,8 +203,8 @@ if [[ ! "$FIRST_USER_NAME" =~ ^[a-z][-a-z0-9_]*$ ]]; then
exit 1
fi
if [[ -n "${APT_PROXY}" ]] && ! curl --silent ${APT_PROXY} >/dev/null ; then
echo "Could not reach APT_PROXY server:" ${APT_PROXY}
if [[ -n "${APT_PROXY}" ]] && ! curl --silent "${APT_PROXY}" >/dev/null ; then
echo "Could not reach APT_PROXY server: ${APT_PROXY}"
exit 1
fi

Loading…
Cancel
Save