grep: do not segfault when -f is used

[git.git] / git-request-pull.sh
diff --git a/git-request-pull.sh b/git-request-pull.sh

index 073a314c8043e0ff30afde65e012e356ff0d186f..fd95beadab6aa01fc3fb26c7d72ba182201f8502 100755 (executable)
--- a/git-request-pull.sh
+++ b/git-request-pull.sh
@@ -4,14 +4,17 @@
  # This file is licensed under the GPL v2, or a later version
  # at the discretion of Linus Torvalds.
  
-USAGE='<commit> <url> [<head>]'
-LONG_USAGE='Summarizes the changes since <commit> to the standard output,
-and includes <url> in the message generated.'
+USAGE='<start> <url> [<end>]'
+LONG_USAGE='Summarizes the changes between two commits to the standard output,
+and includes the given URL in the generated summary.'
  SUBDIRECTORY_OK='Yes'
  OPTIONS_SPEC=
  . git-sh-setup
  . git-parse-remote
  
+GIT_PAGER=
+export GIT_PAGER
+
  base=$1
  url=$2
  head=${3-HEAD}
@@ -25,16 +28,16 @@ headrev=`git rev-parse --verify "$head"^0` || exit
  merge_base=`git merge-base $baserev $headrev` ||
  die "fatal: No commits in common between $base and $head"
  
-url=$(get_remote_url "$url")
  branch=$(git ls-remote "$url" \
         | sed -n -e "/^$headrev refs.heads./{
                 s/^.*   refs.heads.//
                 p
                 q
         }")
+url=$(get_remote_url "$url")
  if [ -z "$branch" ]; then
         echo "warn: No branch of $url is at:" >&2
-       git log --max-count=1 --pretty='format:warn:   %h: %s' $headrev >&2
+       git log --max-count=1 --pretty='tformat:warn:   %h: %s' $headrev >&2
         echo "warn: Are you sure you pushed $head there?" >&2
         echo >&2
         echo >&2
@@ -42,8 +45,6 @@ if [ -z "$branch" ]; then
         status=1
  fi
  
-PAGER=
-export PAGER
  echo "The following changes since commit $baserev:"
  git shortlog --max-count=1 $baserev | sed -e 's/^\(.\)/  \1/'