scripts: do not use optional 'which' command in get-helm installation (#8048)

When installing Helm, the following warning gets printed on the console:

    Downloading https://get.helm.sh/helm-v2.16.6-linux-amd64.tar.gz
    Preparing to install helm and tiller into /usr/local/bin
    helm installed into /usr/local/bin/helm
    tiller installed into /usr/local/bin/tiller
    main: line 178: which: command not found
    Run 'helm init' to configure helm.

The 'which' command is optional, and not always installed on all
environments (like a Fedora container). Instead, use 'command -v' to
detect if the executable is in the $PATH.

Signed-off-by: Niels de Vos <ndevos@redhat.com>
pull/8112/head
Niels de Vos 5 years ago committed by GitHub
parent 59eed4e81f
commit bf12ae3934
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -175,7 +175,7 @@ fail_trap() {
# testVersion tests the installed client to make sure it is working. # testVersion tests the installed client to make sure it is working.
testVersion() { testVersion() {
set +e set +e
HELM="$(which $PROJECT_NAME)" HELM="$(command -v $PROJECT_NAME)"
if [ "$?" = "1" ]; then if [ "$?" = "1" ]; then
echo "$PROJECT_NAME not found. Is $HELM_INSTALL_DIR on your "'$PATH?' echo "$PROJECT_NAME not found. Is $HELM_INSTALL_DIR on your "'$PATH?'
exit 1 exit 1

@ -165,7 +165,7 @@ fail_trap() {
# testVersion tests the installed client to make sure it is working. # testVersion tests the installed client to make sure it is working.
testVersion() { testVersion() {
set +e set +e
HELM="$(which $BINARY_NAME)" HELM="$(command -v $BINARY_NAME)"
if [ "$?" = "1" ]; then if [ "$?" = "1" ]; then
echo "$BINARY_NAME not found. Is $HELM_INSTALL_DIR on your "'$PATH?' echo "$BINARY_NAME not found. Is $HELM_INSTALL_DIR on your "'$PATH?'
exit 1 exit 1

Loading…
Cancel
Save