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