mirror of
https://github.com/community-scripts/ProxmoxVE.git
synced 2026-03-03 18:15:53 +00:00
Remove non-TTY static spinner fallback
Delete the stderr TTY check and the static spinner printf/early return in msg_info. The function now always calls color_spinner and starts the animated spinner in the background, removing the special-case for piped/non-TTY environments and simplifying terminal handling.
This commit is contained in:
@@ -699,15 +699,6 @@ msg_info() {
|
||||
return
|
||||
fi
|
||||
|
||||
# Non-TTY (e.g. running inside lxc-attach | tee): use static spinner character.
|
||||
# The pipe still forwards \r sequences to the host terminal, so the animated
|
||||
# spinner works visually, but backgrounding it is unreliable in some pipe
|
||||
# scenarios. Use a static indicator that clearly shows "in progress".
|
||||
if [[ ! -t 2 ]]; then
|
||||
printf "\r\e[2K %b" "${YW}⠋ ${msg}${CL}"
|
||||
return
|
||||
fi
|
||||
|
||||
color_spinner
|
||||
spinner &
|
||||
SPINNER_PID=$!
|
||||
|
||||
Reference in New Issue
Block a user