From: Daniel Barkalow <barkalow@iabervon.org>
Date: Sun, 16 Mar 2008 23:11:48 GMT
Subject: Re: tracking repository
Message-ID: <alpine.LNX.1.00.0803161904360.19665@iabervon.org>
In-Reply-To: <7vbq5eff3e.fsf@gitster.siamese.dyndns.org>

On Sun, 16 Mar 2008, Junio C Hamano wrote:

> Junio C Hamano <gitster@pobox.com> writes:
> 
> > Daniel Barkalow <barkalow@iabervon.org> writes:
> >
> >> Is "refs/*:refs/*" (mirror everything, including weird stuff) supposed to 
> >> be prohibited?
> >
> > No.  In fact "remote add --mirror" actively creates such.  See my other
> > message about design level issues.
> 
> I think something like this is needed.  It still has an independent issue
> that this is now called by "git remote show" or "git remote prune", and it
> will die with a nonsense "refusing to create" error message, though.
> 
> The error, as far as I can tell, is half about a misconfigured config
> (e.g. "fetch = refs/heads/*:refs/remotes/[]?/*") and half about screwy
> remote repository (e.g. a misnamed "[]?" branch on the remote end can try
> to update a broken "refs/remotes/origin/[]?" even the configuration is a
> perfectly valid "fetch = refs/heads/*:refs/remotes/origin/*").  It may
> make sense to reword the error message to "ignoring" from "refusing" and
> do just that without dying here.  I dunno.

Yeah, I think that's right. (And this patch is also right)

	-Daniel
*This .sig left intentionally blank*