T-Gander/ProxmoxVEDHelperScripts
1
0
Fork 0
Code Issues Pull Requests Actions 5 Packages Projects Releases Wiki Activity

fix(forgejo-runner): revert TTY check, gate on mode being explicitly set

Browse Source 
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.
This commit is contained in:
Cyra
2026-03-29 19:33:17 -07:00
parent 237c0b7f02
commit 17b2229bce
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
ct/forgejo-runner.sh
View File

@@ -63,8 +63,9 @@ function update_script() {
} }
# Fail early if running unattended without required values # Fail early if running unattended without required values
# Only check when there's no TTY (truly non-interactive, e.g. piped or scripted) # mode is only set when the user explicitly passes it (automating);
if [[ -n "${mode:-}" && ! -t 0 ]]; then # bare "bash -c $(curl ...)" leaves mode empty and shows the whiptail menu
if [[ -n "${mode:-}" ]]; then
if [[ -z "${var_forgejo_instance:-}" ]]; then if [[ -z "${var_forgejo_instance:-}" ]]; then
msg_error "var_forgejo_instance is required for unattended installs." msg_error "var_forgejo_instance is required for unattended installs."
exit 1 exit 1