Re: [PATCH] Update "gc" behavior in commit, merge, am, rebase and index-pack
- From
Jeff King <peff@peff.net>
- Date
- May 14, 2012, 20:11 UTC
- Message-ID
- <20120514201100.GA3740@sigill.intra.peff.net>
- In-Reply-To
- <alpine.LFD.2.02.1205121326380.21030@xanadu.home>
On Sat, May 12, 2012 at 01:36:28PM -0400, Nicolas Pitre wrote:
Show 8 quoted lines
> > Instead, we could just inform users that "gc" should be run soon in > > commonly used commands (this patch also reinstates "gc" check in > > commit, which was lost at the sh->C conversion). [1] and [2] can annoy > > users constantly with warnings. This patch shows the warning at most > > once a day. > > I agree with this. However don't bother making this once a day. There > is no harm in warning every time.
I don't mind seeing a warning every time I run "git commit" myself, but aren't there things which run "merge" and "commit" in a loop? For example, "git rebase -m"? Warning 20 times when 20 commits are rebased seems excessive.
-Peff