sha1_file.c: cleanup hdr usage

diff --git a/git-parse-remote.sh b/git-parse-remote.sh
index 144f1701553a1e2f4204a2df15e66437e0670123..5208ee6ce0bb07dd22ad5ca63ee469cb03011164 100755 (executable)
--- a/git-parse-remote.sh
+++ b/git-parse-remote.sh
@@ -10,7 +10,7 @@ get_data_source () {
                echo ''
                ;;
        *)
-               if test "$(git-repo-config --get "remote.$1.url")"
+               if test "$(git-config --get "remote.$1.url")"
                then
                        echo config
                elif test -f "$GIT_DIR/remotes/$1"
@@ -32,7 +32,7 @@ get_remote_url () {
                echo "$1"
                ;;
        config)
-               git-repo-config --get "remote.$1.url"
+               git-config --get "remote.$1.url"
                ;;
        remotes)
                sed -ne '/^URL: */{
@@ -49,8 +49,8 @@ get_remote_url () {
 }
 
 get_default_remote () {
-       curr_branch=$(git-symbolic-ref HEAD | sed -e 's|^refs/heads/||')
-       origin=$(git-repo-config --get "branch.$curr_branch.remote")
+       curr_branch=$(git-symbolic-ref -q HEAD | sed -e 's|^refs/heads/||')
+       origin=$(git-config --get "branch.$curr_branch.remote")
        echo ${origin:-origin}
 }
 
@@ -60,7 +60,7 @@ get_remote_default_refs_for_push () {
        '' | branches)
                ;; # no default push mapping, just send matching refs.
        config)
-               git-repo-config --get-all "remote.$1.push" ;;
+               git-config --get-all "remote.$1.push" ;;
        remotes)
                sed -ne '/^Push: */{
                        s///p
@@ -76,16 +76,39 @@ get_remote_default_refs_for_push () {
 # from get_remote_refs_for_fetch when it deals with refspecs
 # supplied on the command line.  $ls_remote_result has the list
 # of refs available at remote.
+#
+# The first token returned is either "explicit" or "glob"; this
+# is to help prevent randomly "globbed" ref from being chosen as
+# a merge candidate
 expand_refs_wildcard () {
+       remote="$1"
+       shift
+       first_one=yes
+       if test "$#" = 0
+       then
+               echo empty
+               echo >&2 "Nothing specified for fetching with remote.$remote.fetch"
+       fi
        for ref
        do
                lref=${ref#'+'}
                # a non glob pattern is given back as-is.
                expr "z$lref" : 'zrefs/.*/\*:refs/.*/\*$' >/dev/null || {
+                       if test -n "$first_one"
+                       then
+                               echo "explicit"
+                               first_one=
+                       fi
                        echo "$ref"
                        continue
                }
 
+               # glob
+               if test -n "$first_one"
+               then
+                       echo "glob"
+                       first_one=
+               fi
                from=`expr "z$lref" : 'z\(refs/.*/\)\*:refs/.*/\*$'`
                to=`expr "z$lref" : 'zrefs/.*/\*:\(refs/.*/\)\*$'`
                local_force=
@@ -116,15 +139,20 @@ canon_refs_list_for_fetch () {
        if test "$1" = "-d"
        then
                shift ; remote="$1" ; shift
-               set x $(expand_refs_wildcard "$@")
+               set $(expand_refs_wildcard "$remote" "$@")
+               is_explicit="$1"
                shift
                if test "$remote" = "$(get_default_remote)"
                then
-                       curr_branch=$(git-symbolic-ref HEAD | \
+                       curr_branch=$(git-symbolic-ref -q HEAD | \
                            sed -e 's|^refs/heads/||')
-                       merge_branches=$(git-repo-config \
+                       merge_branches=$(git-config \
                            --get-all "branch.${curr_branch}.merge")
                fi
+               if test -z "$merge_branches" && test $is_explicit != explicit
+               then
+                       merge_branches=..this.will.never.match.any.ref..
+               fi
        fi
        for ref
        do
@@ -146,16 +174,12 @@ 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
-               '') remote=HEAD ;;
+               '' | HEAD ) remote=HEAD ;;
                refs/heads/* | refs/tags/* | refs/remotes/*) ;;
                heads/* | tags/* | remotes/* ) remote="refs/$remote" ;;
                *) remote="refs/heads/$remote" ;;
@@ -184,7 +208,7 @@ get_remote_default_refs_for_fetch () {
                echo "HEAD:" ;;
        config)
                canon_refs_list_for_fetch -d "$1" \
-                       $(git-repo-config --get-all "remote.$1.fetch") ;;
+                       $(git-config --get-all "remote.$1.fetch") ;;
        branches)
                remote_branch=$(sed -ne '/#/s/.*#//p' "$GIT_DIR/branches/$1")
                case "$remote_branch" in '') remote_branch=master ;; esac
@@ -258,3 +282,16 @@ resolve_alternates () {
                esac
        done
 }
+
+get_uploadpack () {
+       data_source=$(get_data_source "$1")
+       case "$data_source" in
+       config)
+               uplp=$(git-config --get "remote.$1.uploadpack")
+               echo ${uplp:-git-upload-pack}
+               ;;
+       *)
+               echo "git-upload-pack"
+               ;;
+       esac
+}