Re: [PATCH 3/3] clone: always fetch remote HEAD
- From
Jeff King <peff@peff.net>
- Date
- Jun 6, 2011, 16:11 UTC
- 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:
Show 17 quoted lines
> 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