From b07d54a12e7d42228093d0da7365617defd042e6 Mon Sep 17 00:00:00 2001
From: remph <lhr@disroot.org>
Date: Sat, 26 Oct 2024 18:02:47 +0100
Subject: Fix on dash; some docs/comments clarifications

---
 README.md | 5 +++--
 swob.sh   | 8 ++++----
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/README.md b/README.md
index e53985d..3825d9c 100644
--- a/README.md
+++ b/README.md
@@ -3,8 +3,9 @@ swob (sway+wob helper)
 
 This is a simple/stupid helper script and config files to provide volume and
 brightness controls under Wayland using [wob]. Sway is *not* mandatory (the
-helper script doesn't depend on sway or any wlroots features) but a sway
-config snippet is provided for convenience.
+helper script doesn't depend on sway or any wlroots features); the name is
+simply because sway was the first compositor I configured this for. Other
+compositor configurations are available; see [Installation](#installation).
 
 [wob]: https://github.com/francma/wob
 
diff --git a/swob.sh b/swob.sh
index d0a92b6..bdcdf01 100755
--- a/swob.sh
+++ b/swob.sh
@@ -5,18 +5,18 @@
 wobfifo=${XDG_RUNTIME_DIR:-${TMPDIR:-/tmp}}/wob
 wobini=
 readonly wobfifo
-set -o pipefail -efmu	# `set -e' comes after readonly and pipefail, they
-			# aren't vital enough to kill the script for
+# `set -e' comes after readonly, which isn't vital enough to kill the script for
+set ${BASH_VERSION:+-o pipefail} -efmu
 
 set_wobini() {
 	for dir in ${XDG_CONFIG_HOME:+"$XDG_CONFIG_HOME"} ~/.config /etc; do
 		if test -r "$dir"/swob/wob.ini; then
-			wobini="$dir"/swob/wob.ini
+			wobini=$dir/swob/wob.ini
 			return
 		fi
 	done
 
-	# fallthrough to default: temporary wob.ini(5) standin
+	# fine, I will make my own wob.ini(5)
 	wobini=${XDG_CONFIG_HOME:-~/.config}/swob/wob.ini
 	echo >&2 "$0: no swob/wob.ini found; writing default to $wobini"
 	mkdir -p "${wobini%/*}"
-- 
cgit