On Fri, Oct 25, 2013 at 09:03:41AM +0200, Johan Herland wrote:
> Actually, I don't think there's much refspec stuff to be done here.
> When running "git diff $remote/$branch", there are 3 possible
> - $remote is not a valid remote name, the user probably meant
> something different (like "nested/branch"). The current error message
> is fine.
> - $remote is a valid remote name, but $branch has not (yet) been
> fetched from there. Suggest the user run "git fetch $remote"
> - $remote/$branch is a valid remote-tracking branch. The diff works! No
Right, I think it is the second case we are talking about.
> So, AFAICS, the patch should simply:
> 1. Split the input on the first '/' into $remote/$branch, and use the
> preceding part ($remote) as a potential remote name, and the following
> part ($branch) as a potential branch name. (Although it is
> theoretically possible to have remote names containing slashes, I
> don't think anybody uses them, and we have considered disallowing
> them, mainly because of this very issue: it makes "$remote/$branch"
> parsing (even more) ambiguous)
What I specifically meant is that this breaks with a remote like:
fetch = +refs/heads/*:refs/remotes/bar/*
The correct advice for "bar/branch" is to recommend "git fetch foo", and
the correct advice for "foo/branch" is nothing at all.
I know such config is unusual, but I thought there was a recent push for
us to be accurate about finding the local side of remote tracking
branches, rather than just assuming they start with "$remote". Maybe I
am misremembering, though; I thought it was related to potentially
shifting the default refspecs.
The procedure along those lines would be:
for each remote
for each fetch-refspec in remote
if refspec.rhs contains "refs/remotes/$failed_branch"
recommend "git fetch $remote"
I was just wondering if we had something to make that "does this refspec
contain this ref" part easier.
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html