# [PATCH] State what commit to expect in request-pull

2 messages from 2011-09-06 to 2011-09-06. Participants: Junio C Hamano, Shawn Pearce.
Thread: https://gitlist.dev/t/28317

## Junio C Hamano, 2011-09-06 21:24

Subject: [PATCH] State what commit to expect in request-pull
Message-ID: <7vwrdl749z.fsf@alter.siamese.dyndns.org>
URL: https://gitlist.dev/e/7vwrdl749z.fsf%40alter.siamese.dyndns.org

```
Interestingly, we did say what commit the requesters based their changes
on, but did not give the person who performs a fetch & merge to make sure
he fetched the right branch.

This fixes it.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
---
 git-request-pull.sh     |    1 +
 t/t5150-request-pull.sh |    1 +
 2 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/git-request-pull.sh b/git-request-pull.sh
index fc080cc..9b06a87 100755
--- a/git-request-pull.sh
+++ b/git-request-pull.sh
@@ -71,6 +71,7 @@ git show -s --format='The following changes since commit %H:
 
 are available in the git repository at:' $baserev &&
 echo "  $url $branch" &&
+echo "for you to fetch changes up to $headrev"
 echo &&
 
 git shortlog ^$baserev $headrev &&
diff --git a/t/t5150-request-pull.sh b/t/t5150-request-pull.sh
index 9cc0a42..9d27c43 100755
--- a/t/t5150-request-pull.sh
+++ b/t/t5150-request-pull.sh
@@ -194,6 +194,7 @@ test_expect_success 'pull request format' '
 
 	are available in the git repository at:
 	  URL BRANCH
+	for you to fetch changes up to OBJECT_NAME
 
 	SHORTLOG
 

```

## Shawn Pearce, 2011-09-06 21:58

Subject: Re: [PATCH] State what commit to expect in request-pull
Message-ID: <CAJo=hJsHsVGm1xo4PP416Ek5M+3cc-VZ9LcDBhePNgahcjzB=A@mail.gmail.com>
URL: https://gitlist.dev/e/CAJo%3DhJsHsVGm1xo4PP416Ek5M%2B3cc-VZ9LcDBhePNgahcjzB%3DA%40mail.gmail.com
In-Reply-To: <7vwrdl749z.fsf@alter.siamese.dyndns.org>

```
On Tue, Sep 6, 2011 at 14:24, Junio C Hamano <gitster@pobox.com> wrote:
> Interestingly, we did say what commit the requesters based their changes
> on, but did not give the person who performs a fetch & merge to make sure
> he fetched the right branch.
...
> diff --git a/git-request-pull.sh b/git-request-pull.sh
> @@ -71,6 +71,7 @@ git show -s --format='The following changes since commit %H:
>
>  are available in the git repository at:' $baserev &&
>  echo "  $url $branch" &&
> +echo "for you to fetch changes up to $headrev"
>  echo &&

Better than what we had before. I was sort of hoping for changing the
pull line slightly to include the SHA-1. E.g. suppose instead the line
was:

  echo "  $url $headrev"

and git pull passed the $headrev down to git fetch, which then found
any reference that points exactly at $headrev and fetches that. The
branch name is now lost for the merge summary, but could be guessed
from the advertisement similar to how HEAD is guessed. Git pull could
also then validate that the commit its merging in FETCH_HEAD is what
was passed on the command line.

Then verification of the SHA-1 is automatic by Git, and doesn't
require a human. If the pull request was sent by GPG signed email, and
the MUA checked the signature, the puller has pretty reasonable
assurance over the content they are bringing into their repository.

-- 
Shawn.

```