From: Jeff King <peff@peff.net>
Date: Mon, 23 Nov 2009 19:32:42 GMT
Subject: Re: [PATCH] pack-objects: split implications of --all-progress from progress activation
Message-ID: <20091123193242.GA3140@coredump.intra.peff.net>
In-Reply-To: <alpine.LFD.2.00.0911231221320.2059@xanadu.home>

On Mon, Nov 23, 2009 at 12:43:50PM -0500, Nicolas Pitre wrote:

> Currently the --all-progress flag is used to use force progress display 
> during the writing object phase even if output goes to stdout which is 
> primarily the case during a push operation.  This has the unfortunate 
> side effect of forcing progress display even if stderr is not a 
> terminal.
> 
> Let's introduce the --all-progress-implied argument which has the same 
> intent except for actually forcing the activation of any progress 
> display.  With this, progress display will be automatically inhibited 
> whenever stderr is not a terminal, or full progress display will be 
> included otherwise.  This should let people use 'git push' within a cron 
> job without filling their logs with useless percentage displays.
> 
> Signed-off-by: Nicolas Pitre <nico@fluxnic.net>

Thanks,

Tested-by: Jeff King <peff@peff.net>

-Peff