Re: how to suppress progress percentage in git-push
- From
Jeff King <peff@peff.net>
- Date
- Nov 23, 2009, 16:43 UTC
- Message-ID
- <20091123164319.GA23011@sigill.intra.peff.net>
- In-Reply-To
- <20091123155043.GA28963@machine.or.cz>
On Mon, Nov 23, 2009 at 04:50:43PM +0100, Petr Baudis wrote:
Show 11 quoted lines
> On Mon, Nov 23, 2009 at 10:00:00AM -0500, Jeff King wrote: > > The patch for (1) would look something like what's below. It's simpler, > > but it does change the semantics; anyone who was relying on > > --all-progress to turn on progress unconditionally would need to now > > also use --progress. However, turning on progress unconditionally is > > usually an error (the except is if you are piping output in real-time to > > the user and need to overcome the isatty check). > > I'm actually doing exactly that in the mirrorproj.cgi of Girocco, so I > would be unhappy if I would have to go through creating ptys or whatever > now. Maybe conditioning this by an environment variable?
You wouldn't need to do anything that drastic. You would just need to pass "--progress --all-progress" instead of only --all-progress. But you have provided the data point that such a change would break at least one user.
We could also leave --all-progress as-is and add new option to mean "if you are already doing progress, do all progress".
-Peff