Tools/git-repo
1
0
Fork 0
Code Activity

progress: Make end() idempotent

Browse source 
This fixes the double "done" text on successful interleaved sync.

Bug: 421935613
Change-Id: I4f01418cb0340129a8f0a2a5835f7e3fa6a6b119
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/487081
Reviewed-by: Scott Lee <ddoman@google.com>
Tested-by: Gavin Mak <gavinmak@google.com>
Commit-Queue: Gavin Mak <gavinmak@google.com>
This commit is contained in:
Gavin Mak 2025-06-26 18:08:34 +00:00 • committed by LUCI
parent 82d500eb7a
commit 5d95ba8d85
1 changed files with 5 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

5
progress.py
View file

@ -101,6 +101,7 @@ class Progress:
self._units = units self._units = units
self._elide = elide and _TTY self._elide = elide and _TTY
self._quiet = quiet self._quiet = quiet
self._ended = False
# Only show the active jobs section if we run more than one in parallel. # Only show the active jobs section if we run more than one in parallel.
self._show_jobs = False self._show_jobs = False
@ -211,6 +212,10 @@ class Progress:
self.update(inc=0) self.update(inc=0)
def end(self): def end(self):
if self._ended:
return
self._ended = True
self._update_event.set() self._update_event.set()
if not _TTY or IsTraceToStderr() or self._quiet: if not _TTY or IsTraceToStderr() or self._quiet:
return return