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 /mbc | |
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 'mbc')
-rwxr-xr-x | mbc | 15 |
1 files changed, 6 insertions, 9 deletions
@@ -3,15 +3,15 @@ #{{{ printhelp printhelp () { printf "mbc: mullvad-browser 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 ARG 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 @@ -19,21 +19,18 @@ esac done #{{{ Native Profiles if [ -f ~/.mullvad/mullvadbrowser/profiles.ini ]; then - NATIVE_PROFILES=`grep 'Path=' ~/.mullvad/mullvadbrowser/profiles.ini | cut -d'.' -f2` + NATIVE_PROFILES=$(grep 'Path=' ~/.mullvad/mullvadbrowser/profiles.ini | cut -d'.' -f2) else unset NATIVE_PROFILES fi #}}} #{{{ Add Profiles -if [ -n "$NATIVE_PROFILES" ]; then - INT="\n" -fi PROFILES="$NATIVE_PROFILES" #}}} #{{{ Choose Profiles -CHOSEN=`printf "$PROFILES" | dmenu -l 15 -p "mullvad-browser" $DMENU_ARGS` +CHOSEN=$(printf "%s" "$PROFILES" | dmenu -l 15 -p "mullvad-browser" ${ARG:+"$ARG"}) [ -z "$CHOSEN" ] && exit 2 -exec mullvad-browser -P "$CHOSEN" $URL +exec mullvad-browser -P "$CHOSEN" "$URL" #}}} |