Re: [PATCH 02/12] rev-list: add optional progress reporting

On Fri, Jul 15, 2016 at 11:00:52AM -0700, Junio C Hamano wrote: > Jeff King writes: > > > diff --git a/builtin/rev-list.c b/builtin/rev-list.c > > index b82bcc3..88d95a7 100644 > > --- a/builtin/rev-list.c > > +++ b/builtin/rev-list.c > > @@ -9,6 +9,7 @@ > > #include "log-tree.h" > > #include

Re: [PATCH 02/12] rev-list: add optional progress reporting

Jeff King writes: > diff --git a/builtin/rev-list.c b/builtin/rev-list.c > index b82bcc3..88d95a7 100644 > --- a/builtin/rev-list.c > +++ b/builtin/rev-list.c > @@ -9,6 +9,7 @@ > #include "log-tree.h" > #include "graph.h" > #include "bisect.h" > +#include "progress.h" > > static const char

[PATCH 02/12] rev-list: add optional progress reporting

It's easy to ask rev-list to do a traversal that may take many seconds (e.g., by calling "--objects --all"). In theory you can monitor its progress by the output you get to stdout, but this isn't always easy. Some operations, like "--count", don't make any output until the end. And some callers,