Skip to content

Commit ce46915

Browse files
committed
install.sh: Prefix global variables
1 parent d208a66 commit ce46915

File tree

1 file changed

+31
-31
lines changed

1 file changed

+31
-31
lines changed

install.sh

Lines changed: 31 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ main() {
1919
parse_args "$@"
2020

2121
local _arch
22-
_arch="${ARCH:-$(get_architecture)}" || exit $?
22+
_arch="${_ZOXIDE_ARCH:-$(get_architecture)}" || exit $?
2323
assert_nz "${_arch}" "arch"
2424
echo "Detected architecture: ${_arch}"
2525

@@ -54,44 +54,44 @@ main() {
5454
esac
5555

5656
# Install binary.
57-
ensure try_sudo mkdir -p -- "${BIN_DIR}"
58-
ensure try_sudo cp -- "${_bin_name}" "${BIN_DIR}/${_bin_name}"
59-
ensure try_sudo chmod +x "${BIN_DIR}/${_bin_name}"
60-
echo "Installed zoxide to ${BIN_DIR}"
57+
ensure try_sudo mkdir -p -- "${_ZOXIDE_BIN_DIR}"
58+
ensure try_sudo cp -- "${_bin_name}" "${_ZOXIDE_BIN_DIR}/${_bin_name}"
59+
ensure try_sudo chmod +x "${_ZOXIDE_BIN_DIR}/${_bin_name}"
60+
echo "Installed zoxide to ${_ZOXIDE_BIN_DIR}"
6161

6262
# Install manpages.
63-
ensure try_sudo mkdir -p -- "${MAN_DIR}/man1"
64-
ensure try_sudo cp -- "man/man1/"* "${MAN_DIR}/man1/"
65-
echo "Installed manpages to ${MAN_DIR}"
63+
ensure try_sudo mkdir -p -- "${_ZOXIDE_MAN_DIR}/man1"
64+
ensure try_sudo cp -- "man/man1/"* "${_ZOXIDE_MAN_DIR}/man1/"
65+
echo "Installed manpages to ${_ZOXIDE_MAN_DIR}"
6666

6767
# Print success message and check $PATH.
6868
echo ""
6969
echo "zoxide is installed!"
70-
if ! echo ":${PATH}:" | grep -Fq ":${BIN_DIR}:"; then
71-
echo "Note: ${BIN_DIR} is not on your \$PATH. zoxide will not work unless it is added to \$PATH."
70+
if ! echo ":${PATH}:" | grep -Fq ":${_ZOXIDE_BIN_DIR}:"; then
71+
echo "Note: ${_ZOXIDE_BIN_DIR} is not on your \$PATH. zoxide will not work unless it is added to \$PATH."
7272
fi
7373
}
7474

7575
# Parse the arguments passed and set variables accordingly.
7676
parse_args() {
77-
BIN_DIR_DEFAULT="${HOME}/.local/bin"
78-
MAN_DIR_DEFAULT="${HOME}/.local/share/man"
79-
SUDO_DEFAULT="sudo"
77+
_ZOXIDE_BIN_DIR_DEFAULT="${HOME}/.local/bin"
78+
_ZOXIDE_MAN_DIR_DEFAULT="${HOME}/.local/share/man"
79+
_ZOXIDE_SUDO_DEFAULT="sudo"
8080

81-
BIN_DIR="${BIN_DIR_DEFAULT}"
82-
MAN_DIR="${MAN_DIR_DEFAULT}"
83-
SUDO="${SUDO_DEFAULT}"
81+
_ZOXIDE_BIN_DIR="${_ZOXIDE_BIN_DIR_DEFAULT}"
82+
_ZOXIDE_MAN_DIR="${_ZOXIDE_MAN_DIR_DEFAULT}"
83+
_ZOXIDE_SUDO="${_ZOXIDE_SUDO_DEFAULT}"
8484

8585
while [ "$#" -gt 0 ]; do
8686
case "$1" in
87-
--arch) ARCH="$2" && shift 2 ;;
88-
--arch=*) ARCH="${1#*=}" && shift 1 ;;
89-
--bin-dir) BIN_DIR="$2" && shift 2 ;;
90-
--bin-dir=*) BIN_DIR="${1#*=}" && shift 1 ;;
91-
--man-dir) MAN_DIR="$2" && shift 2 ;;
92-
--man-dir=*) MAN_DIR="${1#*=}" && shift 1 ;;
93-
--sudo) SUDO="$2" && shift 2 ;;
94-
--sudo=*) SUDO="${1#*=}" && shift 1 ;;
87+
--arch) _ZOXIDE_ARCH="$2" && shift 2 ;;
88+
--arch=*) _ZOXIDE_ARCH="${1#*=}" && shift 1 ;;
89+
--bin-dir) _ZOXIDE_BIN_DIR="$2" && shift 2 ;;
90+
--bin-dir=*) _ZOXIDE_BIN_DIR="${1#*=}" && shift 1 ;;
91+
--man-dir) _ZOXIDE_MAN_DIR="$2" && shift 2 ;;
92+
--man-dir=*) _ZOXIDE_MAN_DIR="${1#*=}" && shift 1 ;;
93+
--sudo) _ZOXIDE_SUDO="$2" && shift 2 ;;
94+
--sudo=*) _ZOXIDE_SUDO="${1#*=}" && shift 1 ;;
9595
-h | --help) usage && exit 0 ;;
9696
*) err "Unknown option: $1" ;;
9797
esac
@@ -119,9 +119,9 @@ ${_text_heading}Usage:${_text_reset}
119119
120120
${_text_heading}Options:${_text_reset}
121121
--arch Override the architecture identified by the installer [current: ${_arch}]
122-
--bin-dir Override the installation directory [default: ${BIN_DIR_DEFAULT}]
123-
--man-dir Override the manpage installation directory [default: ${MAN_DIR_DEFAULT}]
124-
--sudo Override the command used to elevate to root privileges [default: ${SUDO_DEFAULT}]
122+
--bin-dir Override the installation directory [default: ${_ZOXIDE_BIN_DIR_DEFAULT}]
123+
--man-dir Override the manpage installation directory [default: ${_ZOXIDE_MAN_DIR_DEFAULT}]
124+
--sudo Override the command used to elevate to root privileges [default: ${_ZOXIDE_SUDO_DEFAULT}]
125125
-h, --help Print help"
126126
}
127127

@@ -176,19 +176,19 @@ try_sudo() {
176176
fi
177177

178178
need_sudo
179-
"${SUDO}" "$@"
179+
"${_ZOXIDE_SUDO}" "$@"
180180
}
181181

182182
need_sudo() {
183-
if ! check_cmd "${SUDO}"; then
183+
if ! check_cmd "${_ZOXIDE_SUDO}"; then
184184
err "\
185-
could not find the command \`${SUDO}\` needed to get permissions for install.
185+
could not find the command \`${_ZOXIDE_SUDO}\` needed to get permissions for install.
186186
187187
If you are on Windows, please run your shell as an administrator, then rerun this script.
188188
Otherwise, please run this script as root, or install \`sudo\`."
189189
fi
190190

191-
if ! "${SUDO}" -v; then
191+
if ! "${_ZOXIDE_SUDO}" -v; then
192192
err "sudo permissions not granted, aborting installation"
193193
fi
194194
}

0 commit comments

Comments
 (0)