Revert "Allow branch.*.merge to talk about remote tracking branches."
authorJunio C Hamano <junkio@cox.net>
Sun, 4 Feb 2007 00:23:38 +0000 (16:23 -0800)
committerJunio C Hamano <junkio@cox.net>
Mon, 5 Feb 2007 00:58:30 +0000 (16:58 -0800)
This reverts commit 80c797764a6b6a373f0f1f47d7f56b0d950418a9.

Back when I committed this, it seemed to be a good idea. People
who always use remote tracking branches can optionally use the
local name they happen to use to specify what to merge, which meant
that I did not have to teach them why we use the name at the remote
side every time they are confused.

But allowing it seems to break other people's scripts. The real
solution is not to allow more ways to express the same thing, but
to educate people to use the right syntax.

Signed-off-by: Junio C Hamano <junkio@cox.net>
git-parse-remote.sh
index 3e783b7b056de113584e526835a23fb05dfdcfce..5208ee6ce0bb07dd22ad5ca63ee469cb03011164 100755 (executable)
@@ -174,12 +174,8 @@ canon_refs_list_for_fetch () {
                else
                        for merge_branch in $merge_branches
                        do
-                           if  test "$remote" = "$merge_branch" ||
-                               test "$local" = "$merge_branch"
-                           then
-                                   dot_prefix=
-                                   break
-                           fi
+                           [ "$remote" = "$merge_branch" ] &&
+                           dot_prefix= && break
                        done
                fi
                case "$remote" in