From: Junio C Hamano Date: Fri, 17 Oct 2008 23:57:53 GMT Subject: Re: "checkout --track -b" broken? (with suggested fix) Message-ID: <7vskqug4fy.fsf@gitster.siamese.dyndns.org> In-Reply-To: Daniel Barkalow writes: > On Thu, 16 Oct 2008, Junio C Hamano wrote: > >> The patch to branch.c is a quick fix for this issue. The resulting code >> passes all the tests, but I am not very proud of hardcoding the "HEAD" in >> the code. There must be a better way to do this. > > I agree with the change to the test. I think it would be better to > hard-code "refs/heads/" instead of "HEAD", and I feel like we must have a > "is this ref name a branch?" function, if only because someone could stick > "refs/tags/foo" in HEAD, and we should still say it's not something you > could track, despite it being something different from "HEAD". But you can track things under refs/remotes/, so...