On Tue, 25 Mar 2008, Junio C Hamano wrote:
> Daniel Barkalow <barkalow@xxxxxxxxxxxx> writes:
>
> > It turns out that .git/branches/foo should fetch into refs/heads/foo.
> >
> > Signed-off-by: Daniel Barkalow <barkalow@xxxxxxxxxxxx>
> > ---
> > We still don't have a test for that bit of the behavior, so I could have
> > it still doing the wrong thing. But it at least should do what I think
> > people want.
> >
> > Andrew, could you give this a try, on top of current master (or, for that
> > matter, any released version that doesn't work), and let me know if it
> > does the wrong thing?
>
> To give bystanders a bit of context, Andrew had an old Cogito-style
>
> .git/branches/foo
>
> that had a URL (and no #frag) in it. "git fetch foo" should have fetched
> from the 'master' branch of the named repository and stored it in his
> refs/heads/foo, and git up to 1.5.3.X series do so correctly, but we had
> this uncaught regression in 1.5.4 (the support for "branches" file is not
> deprecated, even though I personally do not see a reason to use it in new
> repositories).
>
> Now, Daniel, I was independently fixing this one and had a question about
> this code.
>
> What is the deal with this "slash" magic? It appears that when you are
> fed:
>
> git fetch foo/bar
>
> you look up ".git/branches/foo" for $URL and #$branch, then use "$URL/bar"
> as the URL and $branch (or 'master' if you did not see '#') as the branch
> to fetch from, and store it in "foo/bar" locally. Is that really what
> should have happened? I am reasonably sure about "fetching from $URL/bar"
> part, but I am unsure about "store in foo/bar" part.
>
> Any Cogito survivers out there who knows how this was supposed to have
> worked? We can go back and look at git-fetch.sh (in contrib/examples),
> but I'd rather be lazy ;-)
This code actually is from Linus, nearly 2 years ago, converting
git-push.sh; I then moved it from builtin-push.c to remote.c and later
added support to remote.c for the fetch side aspects while leaving that
part of the code alone. Of course, I didn't realize at any point that, as
a fetch configuration, a branches configuration ever stored things
anywhere, and it looks like it didn't with a slash. (See line 115 of
755225de:git-parse-remote.sh)
So I guess there should be:
> > add_url_alias(remote, p);
> > - add_fetch_refspec(remote, branch);
if (!slash)
> > + strbuf_addf(&branch, ":refs/heads/%s", remote->name);
> > + add_fetch_refspec(remote, strbuf_detach(&branch, 0));
> > remote->fetch_tags = 1; /* always auto-follow */
> > }
-Daniel
*This .sig left intentionally blank*
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
