From: Jeff King <peff@peff.net>
Date: Mon, 06 Jun 2011 16:11:43 GMT
Subject: Re: [PATCH 3/3] clone: always fetch remote HEAD
Message-ID: <20110606161143.GC11485@sigill.intra.peff.net>
In-Reply-To: <7vaadvbwpw.fsf@alter.siamese.dyndns.org>

On Sun, Jun 05, 2011 at 06:00:11PM -0700, Junio C Hamano wrote:

> Junio C Hamano <gitster@pobox.com> writes:
> 
> > Jeff King <peff@peff.net> writes:
> >
> >> I did some more digging. I don't think the problem is with HEAD at all,
> >> but rather with asking for more than one ref at all.
> >> ...
> >> Does that make sense? If so, then I think the right fix is for testgit
> >> ti ignore all imports after the first one (since the first one will have
> >> done all available refs). And the patch is:
> >
> > After reading Sverre's answer, I think this makes sense as a short-term
> > workaround.  Shall we squah it in to 3/3?
> 
> I'll queue it separately as a tentative commit (below) and leave it in
> 'pu' for now. Please give it a better description so that we can move the
> fix forward.

I'll try the nicer fix Sverre mentioned and post a real patch. I wanted
to add a test for multiple refs to the test suite, but I got bogged down
by other breakages. Some of which I think are the fault of testgit, but
one of which I think may be a problem in the transport-helper code.

I gave up in frustration after many hours of dealing with it on Friday,
but I'll give it another go today. :)

-Peff