From c74f656694e084fdf748f7ac86aa4b2361cbe1ed Mon Sep 17 00:00:00 2001
From: Craig Jennings <c@cjennings.net>
Date: Fri, 23 Jan 2026 18:02:52 -0600
Subject: Fix set -e compatibility in lib functions

- Replace [[ ]] && error pattern with if/then/fi
- Pattern causes exit when condition is false with set -e
- Fixed in: common.sh, config.sh, disk.sh, install-archzfs

Discovered during VM testing: the && short-circuit returns exit
code 1 when condition is false, triggering set -e to abort.
---
 custom/lib/disk.sh | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

(limited to 'custom/lib/disk.sh')

diff --git a/custom/lib/disk.sh b/custom/lib/disk.sh
index ea8c402..fa3dbd2 100644
--- a/custom/lib/disk.sh
+++ b/custom/lib/disk.sh
@@ -148,7 +148,9 @@ select_disks() {
         read -rp "Enter disk path(s) separated by space: " selected
     fi
 
-    [[ -z "$selected" ]] && error "No disk selected"
+    if [[ -z "$selected" ]]; then
+        error "No disk selected"
+    fi
 
     # Extract just the device paths (remove size/model info)
     SELECTED_DISKS=()
-- 
cgit v1.2.3