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 /brc | |
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 'brc')
-rwxr-xr-x | brc | 21 |
1 files changed, 7 insertions, 14 deletions
@@ -2,16 +2,16 @@ #{{{ printhelp printhelp () { - printf "lwc: librewolf menu interface.\n" - printf " -u URL ) pass URL to browser\n" - printf " -d ARGS ) pass ARGS to menu command\n" + printf "brc: brave profile menu interface.\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,14 @@ esac done #{{{ Native Profiles if [ -d ~/.local/share/brave_profiles ]; then - NATIVE_PROFILES=`ls -1 ~/.local/share/brave_profiles` + PROFILES=$(ls -1 ~/.local/share/brave_profiles) else - unset NATIVE_PROFILES + unset PROFILES fi #}}} -#{{{ Add Profiles -if [ -n "$NATIVE_PROFILES" -a -n "$FLATPAK_PROFILES" ]; then - INT="\n" -fi -PROFILES="$NATIVE_PROFILES$INT$FLATPAK_PROFILES" -#}}} - #{{{ Choose Profiles -CHOSEN=`printf "$PROFILES" | dmenu -l 15 -p brave $DMENU_ARGS` +CHOSEN=$(printf "%s" "$PROFILES" | dmenu -l 15 -p brave ${ARG:+"$ARG"}) [ -z "$CHOSEN" ] && exit 2 brave-profiles "$CHOSEN" "$URL" #}}} |