From 95a56ef4f004f3eaa322393ae3be5fc50d11ff2c Mon Sep 17 00:00:00 2001
From: Justin Oros <JustinOros@gmail.com>
Date: Thu, 16 Apr 2026 10:26:28 -0700
Subject: [PATCH] fix usermod group assignment to use if blocks instead of &&
 chain

---
 hub/setup-hub.sh | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/hub/setup-hub.sh b/hub/setup-hub.sh
index 5afb44e..c6801d1 100644
--- a/hub/setup-hub.sh
+++ b/hub/setup-hub.sh
@@ -78,10 +78,14 @@ else
 
     ADDED_TO_GROUP=false
     if getent group sudo >/dev/null 2>&1; then
-        usermod -aG sudo armbian && ADDED_TO_GROUP=true
+        if usermod -aG sudo armbian 2>/dev/null; then
+            ADDED_TO_GROUP=true
+        fi
     fi
     if [ "$ADDED_TO_GROUP" = false ] && getent group wheel >/dev/null 2>&1; then
-        usermod -aG wheel armbian && ADDED_TO_GROUP=true
+        if usermod -aG wheel armbian 2>/dev/null; then
+            ADDED_TO_GROUP=true
+        fi
     fi
     if [ "$ADDED_TO_GROUP" = false ]; then
         warn "Neither sudo nor wheel group found — armbian user has no sudo access."