From 17b2229bceb6945074df1cdedec445554cbc44c3 Mon Sep 17 00:00:00 2001 From: Cyra Date: Sun, 29 Mar 2026 19:33:17 -0700 Subject: [PATCH] fix(forgejo-runner): revert TTY check, gate on mode being explicitly set mode is only set when the user is automating (e.g. mode=default with var_* overrides). The default bash -c curl invocation leaves mode empty and shows the whiptail menu, so this check won't affect interactive use. --- ct/forgejo-runner.sh | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ct/forgejo-runner.sh b/ct/forgejo-runner.sh index 85faf97c..77e02fe2 100644 --- a/ct/forgejo-runner.sh +++ b/ct/forgejo-runner.sh @@ -63,8 +63,9 @@ function update_script() { } # Fail early if running unattended without required values -# Only check when there's no TTY (truly non-interactive, e.g. piped or scripted) -if [[ -n "${mode:-}" && ! -t 0 ]]; then +# mode is only set when the user explicitly passes it (automating); +# bare "bash -c $(curl ...)" leaves mode empty and shows the whiptail menu +if [[ -n "${mode:-}" ]]; then if [[ -z "${var_forgejo_instance:-}" ]]; then msg_error "var_forgejo_instance is required for unattended installs." exit 1