summaryrefslogtreecommitdiff
path: root/tessen
diff options
context:
space:
mode:
authorAyush Agarwal <ayush@fastmail.in>2022-01-16 18:51:40 +0530
committerAyush Agarwal <ayush@fastmail.in>2022-01-16 21:15:53 +0530
commitca9d48641b0e34bc486cba2dfd0c24cd2ecc9b4e (patch)
treee6148163bd4bb21278c5e391eb4c28923f07c5b2 /tessen
parentedea7a802c93b1347a28b26efe8687b2fa38a9ca (diff)
refactor: change the find_backend function
Recently, I realized that bash local variables aren't actually local and use dynamic scoping. Renaming `backend` to `dmbd` is just a precaution.
Diffstat (limited to 'tessen')
-rwxr-xr-xtessen13
1 files changed, 7 insertions, 6 deletions
diff --git a/tessen b/tessen
index 1e08a19..d92d5c6 100755
--- a/tessen
+++ b/tessen
@@ -380,14 +380,15 @@ get_default_opts() {
}
find_backend() {
- local backend
- for backend in "${tsn_known_backends[@]}"; do
- if is_installed "${backend}"; then
- printf "%s" "${backend}"
+ local dmbd
+
+ for dmbd in "${tsn_known_backends[@]}"; do
+ if is_installed "$dmbd"; then
+ printf "%s" "$dmbd"
return 0
fi
done
- _die "Please use '--backend' option or 'TESSEN_BACKEND' environment variable to configure installed 'dmenu' compatible application to use ${0##*/}"
+ _die "%s\n" "unable to find a 'dmenu' compatible application"
}
validate_cliptime() {
@@ -471,7 +472,7 @@ main() {
unset -v _opt
if [[ -z "${tsn_backend}" ]]; then
- tsn_backend=$(find_backend)
+ tsn_backend="$(find_backend)"
fi
setup_backend "${tsn_backend}"