Re: [PATCH 1/2] send-pack: cluster ref status reporting

From: Jeff King <peff@peff.net>
Date: Nov 21, 2007, 07:39 UTC
Message-ID: <20071121073938.GB10742@sigill.intra.peff.net>
In-Reply-To: <7vwsscgie2.fsf@gitster.siamese.dyndns.org>

On Tue, Nov 20, 2007 at 11:36:05PM -0800, Junio C Hamano wrote:

> Heh, at least I've privately queued it at the tip of
> jk/send-pack for review but not merged it to 'next' yet, after
> fixing it up (the last round I also fixed up so that was partly
> why I noticed -- the patch did not apply).

OK, good. I should have noticed because I also ran across the same patch failure when I did a rebase earlier today, but I didn't consider that the mistake had slipped out of my repo. :) Sorry for the confusion.

> The idea feels sound, and code under cursory look was fine.

I think it is probably worth applying then. It is slightly more useful to the user, and I haven't been able to see any downside.

-Peff