From d9e53d5a16a52e427b35c4f7afb7e989e789ffe1 Mon Sep 17 00:00:00 2001 From: "CanbiZ (MickLesk)" <47820557+MickLesk@users.noreply.github.com> Date: Mon, 2 Mar 2026 09:26:24 +0100 Subject: [PATCH] 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. --- misc/core.func | 9 --------- 1 file changed, 9 deletions(-) diff --git a/misc/core.func b/misc/core.func index f4238b1a5..9a4c52f71 100644 --- a/misc/core.func +++ b/misc/core.func @@ -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=$!