Re: [PATCH] repack.c: rename and unlink pack file if it exists

From: Jeff King <peff@peff.net>
Date: Feb 5, 2014, 21:09 UTC
Message-ID: <20140205210954.GA24142@sigill.intra.peff.net>
In-Reply-To: <xmqq7g992ror.fsf@gitster.dls.corp.google.com>

On Wed, Feb 05, 2014 at 01:08:36PM -0800, Junio C Hamano wrote:

Show 11 quoted lines

> Jeff King <peff@peff.net> writes:
> 
> > ... So the fact that this
> > bug exists doesn't really produce any user-visible behavior, and
> > hopefully post-release we would drop the code entirely, and the test
> > would have no reason to exist.
> 
> Heh, thanks, and I agree with the reasoning for the longer-term
> direction.  Perhaps I can/should hold it off that minimal fix-up
> patch from -rc3, then?  I am on the fence but I already started my
> today's integration cycle _with_ the fix merged to 'master', so...

I would say leave the fix for -rc3, but not worry about the test.

-Peff