diff options
author | zachir <zachir@librem.one> | 2025-08-30 01:09:18 -0500 |
---|---|---|
committer | zachir <zachir@librem.one> | 2025-08-30 01:09:18 -0500 |
commit | f28d694879239d5d9c319185a278236d317a49b6 (patch) | |
tree | 3f684e05c6372c8f0fc0b9f126a184f3aaad68d4 /qbc | |
parent | 7d3cbe8f708772e40608260590b717dea6735908 (diff) |
Make sh scrips POSIX compliant
Using shellcheck, I went through all of them to make them standards
compliant. I also tested as many as I could.
Diffstat (limited to 'qbc')
-rwxr-xr-x | qbc | 18 |
1 files changed, 9 insertions, 9 deletions
@@ -3,36 +3,36 @@ #{{{ printhelp printhelp () { printf "qbc: qutebrowser menu interface.\n" - printf " -u URL ) pass URL to browser\n" - printf " -d ARGS ) pass ARGS to menu command\n" + printf " -u URL ) pass URL to browser\n" + printf " -d ARG ) pass ARGS to menu command\n" exit } #}}} #{{{ getopts while getopts "hd:u:" o; do case "${o}" in - d) DMENU_ARGS="$OPTARG" ;; + d) ARG="$OPTARG" ;; u) URL="$OPTARG" ;; *) printhelp ;; esac done #}}} #{{{ Get Sessions -SESSIONS=`qbpm list` -[ -z "$SESSIONS" ] && qutebrowser $@ && exit +SESSIONS=$(qbpm list) +[ -z "$SESSIONS" ] && qutebrowser "$@" && exit #}}} #{{{ Select Session -CHOSEN=`printf "default\n$SESSIONS" | dmenu -l 15 -p "qutebrowser" $DMENU_ARGS` +CHOSEN=$(printf "default\n%s" "$SESSIONS" | dmenu -l 15 -p "qutebrowser" ${ARG:+"$ARG"}) [ -z "$CHOSEN" ] && exit 2 if [ -n "$CHOSEN" ]; then if [ "$CHOSEN" = "default" ]; then - qutebrowser $URL + qutebrowser "$URL" else - if [ -z `echo "$SESSIONS" | grep "$CHOSEN"` ]; then + if echo "$SESSIONS" | { ! grep -q "$CHOSEN" ; }; then qbpm new "$CHOSEN" fi - qbpm launch "$CHOSEN" $URL + qbpm launch "$CHOSEN" "$URL" #qutebrowser -B "$HOME/.local/share/$CHOSEN" fi fi |