From: Lennart Poettering <lennart@poettering.net>
Date: Mon, 6 Jan 2025 21:36:50 +0000 (+0100)
Subject: pretty-print: drop extra ';' from progress reporting end sequence
X-Git-Tag: v258-rc1~1689
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=20785351b1c76599b2dd413d2ce259bdbed38b21;p=thirdparty%2Fsystemd.git

pretty-print: drop extra ';' from progress reporting end sequence

This corrects the closing sequence for the ConEmu progress reporting
final sequence. We by mistake sent two final ;;, where only one was
expected. The terminals I tested this with didn't care, but Ghostty
apparently does. Let's fix things and generate the closing sequence as
per doc:

https://conemu.github.io/en/AnsiEscapeCodes.html#ConEmu_specific_OSC
---

diff --git a/src/shared/pretty-print.c b/src/shared/pretty-print.c
index 1b735625847..d8a55376585 100644
--- a/src/shared/pretty-print.c
+++ b/src/shared/pretty-print.c
@@ -556,7 +556,7 @@ void clear_progress_bar_unbuffered(const char *prefix) {
                       stderr);
         else
                 /* Undo Windows Terminal progress indication again. */
-                fputs(ANSI_OSC "9;4;0;;" ANSI_ST
+                fputs(ANSI_OSC "9;4;0;" ANSI_ST
                       ANSI_ERASE_TO_END_OF_LINE, stderr);
 
         fputc('\r', stderr);