Re: [PATCH] builtin-remote: better handling of multiple remote HEADs

On Sat, Feb 14, 2009 at 04:15:48PM -0500, Jeff King wrote:

On Sat, Feb 14, 2009 at 03:21:30PM -0500, Daniel Barkalow wrote:

quoted

I haven't checked lately, but I think that what's actually needed is to 
have the locate_head() function notice if the struct ref for HEAD actually 
has the symref field non-NULL, and report that as the unambiguous answer. 

Indeed. Something like the patch below works (on top of Jay's patches).
[...]
I will try to make something a little neater later today.

Here is a cleaner series. It depends on all of Jay's remote patches,
including the set-head one. I can resend once the dust is a little more
settled on those patches.

The first two are prep for adding a test in 5/5:

   1/5: test scripts: refactor start_httpd helper
   2/5: add basic http clone/fetch tests

These ones are code cleanup for 5/5:

   3/5: refactor find_refs_by_name to accept const list
   4/5: remote: refactor guess_remote_head

And this is the useful one.

   5/5: remote: use exact HEAD lookup if it is available

-Peff