ab/only-single-progress-at-once (was: What's cooking in git.git (Dec 2021, #05; Thu, 23))

From: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date: Dec 26, 2021, 17:24 UTC
Message-ID: <211226.86y247ff9b.gmgdl@evledraar.gmail.com>
In-Reply-To: <xmqqsfui6g7q.fsf@gitster.g>

On Thu, Dec 23 2021, Junio C Hamano wrote:

Show 17 quoted lines

> * 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@gmail.com>

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@gmail.com/

And the old v6 you've got here is at: https://lore.kernel.org/git/cover-v6-0.8-00000000000-20211102T122507Z-avarab@gmail.com/