On Thu, Dec 23 2021, Junio C Hamano wrote: > * ab/only-single-progress-at-once (2021-11-03) 8 commits > - progress.c: add & assert a "global_progress" variable > - various *.c: use isatty(0|2), not isatty(STDIN_FILENO|STDERR_FILENO) > - pack-bitmap-write.c: don't return without stop_progress() > - progress.c: add temporary variable from progress struct > - progress.c tests: test some invalid usage > - progress.c tests: make start/stop commands on stdin > - progress.c test helper: add missing braces > - leak tests: fix a memory leaks in "test-progress" helper > > Further tweaks on progress API. > > Needs review. > The last three rounds has seen little reaction, even though earlier > round saw a lot of responses. The latest round needs a serious > review or at least Acks from past commentors. > source: <cover-v6-0.8-00000000000-20211102T122507Z-avarab@xxxxxxxxx> This is the v6, but I submitted a v7 on December 17th that should address the outstanding questions about this series, i.e. it ejected the controversial last patch adding the BUG(). I can pursue that later, which should make its review easier. I inadvertently broke threading in the v7 submission, sorry about that, but here it is: https://lore.kernel.org/git/cover-v7-0.7-00000000000-20211217T041945Z-avarab@xxxxxxxxx/ And the old v6 you've got here is at: https://lore.kernel.org/git/cover-v6-0.8-00000000000-20211102T122507Z-avarab@xxxxxxxxx/
