Skip to content

Commit 7e93e49

Browse files
committed
fix: remove local keyword from top-level script context
The setup-tools script runs chainsaw installation at the top level, not inside a function, so local is invalid in bash strict mode.
1 parent 060e37e commit 7e93e49

File tree

1 file changed

+13
-13
lines changed

1 file changed

+13
-13
lines changed

tools/setup-tools

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -365,21 +365,21 @@ if [[ "${SKIP_TOOLS}" == "false" ]]; then
365365
else
366366
log_info "Installing Chainsaw ${CHAINSAW_VERSION}..."
367367
if prompt_continue; then
368-
local ver="${CHAINSAW_VERSION#v}"
369-
local os_name; os_name="$(uname -s | tr '[:upper:]' '[:lower:]')"
370-
local arch; arch="$(uname -m)"
371-
case "${arch}" in
372-
x86_64) arch="amd64" ;;
373-
aarch64|arm64) arch="arm64" ;;
368+
CHAINSAW_VER="${CHAINSAW_VERSION#v}"
369+
CHAINSAW_OS="$(uname -s | tr '[:upper:]' '[:lower:]')"
370+
CHAINSAW_ARCH="$(uname -m)"
371+
case "${CHAINSAW_ARCH}" in
372+
x86_64) CHAINSAW_ARCH="amd64" ;;
373+
aarch64|arm64) CHAINSAW_ARCH="arm64" ;;
374374
esac
375-
local tar="chainsaw_${os_name}_${arch}.tar.gz"
376-
local url="https://github.com/kyverno/chainsaw/releases/download/v${ver}/${tar}"
377-
local tmp; tmp="$(mktemp -d)"
378-
curl -fsSL -o "${tmp}/${tar}" "${url}"
379-
tar -xzf "${tmp}/${tar}" -C "${tmp}"
380-
sudo mv "${tmp}/chainsaw" /usr/local/bin/chainsaw
375+
CHAINSAW_TAR="chainsaw_${CHAINSAW_OS}_${CHAINSAW_ARCH}.tar.gz"
376+
CHAINSAW_URL="https://github.com/kyverno/chainsaw/releases/download/v${CHAINSAW_VER}/${CHAINSAW_TAR}"
377+
CHAINSAW_TMP="$(mktemp -d)"
378+
curl -fsSL -o "${CHAINSAW_TMP}/${CHAINSAW_TAR}" "${CHAINSAW_URL}"
379+
tar -xzf "${CHAINSAW_TMP}/${CHAINSAW_TAR}" -C "${CHAINSAW_TMP}"
380+
sudo mv "${CHAINSAW_TMP}/chainsaw" /usr/local/bin/chainsaw
381381
sudo chmod +x /usr/local/bin/chainsaw
382-
rm -rf "${tmp}"
382+
rm -rf "${CHAINSAW_TMP}"
383383
log_success "Chainsaw installed"
384384
fi
385385
fi

0 commit comments

Comments
 (0)