On Sun, 16 Mar 2008, Junio C Hamano wrote: > Junio C Hamano <gitster@xxxxxxxxx> writes: > > > Daniel Barkalow <barkalow@xxxxxxxxxxxx> 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* -- 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