From 3e75c51d1861b7807b8904389499faa2fb9bc47d Mon Sep 17 00:00:00 2001 From: Claude Date: Mon, 10 Aug 2026 14:53:44 +0000 Subject: [PATCH] Fix "local: can only be used in a function" crash in the category menu MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The dynamic checklist-sizing code added in the previous commit used `local` for its variables, but the category menu loop it lives in is top-level script code, not inside a function — `local` only works inside one. Confirmed live: this broke the whiptail menu outright on first `sudo ./setup.sh` run after pulling ("only be used in a function", then an unbound-variable error under set -u since the assignment before it never ran). Drop `local`; these are the same kind of plain loop-scoped variables every other var in this loop (CHOSEN_CAT, SVCS, CHOICE, SELECTED) already is. --- setup.sh | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/setup.sh b/setup.sh index 020616c..879a3ef 100755 --- a/setup.sh +++ b/setup.sh @@ -390,11 +390,10 @@ while true; do # terminal can show (tput lines, falling back to a conservative 24 # for a non-terminal/unknown size) so this can't request a dialog # taller than the screen. - local _term_lines _list_h _box_h _term_lines="$(tput lines 2>/dev/null