threads / discuss / 36277

Git-hooks pre-push script does not receive input on stdin

Subject: Git-hooks pre-push script does not receive input on stdin

## tl;dr

3 messages between Mar 23, 2014 and Mar 23, 2014.

replies: 2people: 2as markdown or json

David Cowden· Mar 23, 2014, 00:44 UTC · lore
http://stackoverflow.com/questions/22585091/git-hooks-pre-push-script-does-not-receive-input-via-stdin
Is this a bug in git?  Or am I just a shell noob?
Thanks in advance,
David
Torsten Bögershausen· Mar 23, 2014, 07:19 UTC · re: David Cowden · lore

Re: Git-hooks pre-push script does not receive input on stdin

On 2014-03-23 01.44, David Cowden wrote:
Show 7 quoted lines
> http://stackoverflow.com/questions/22585091/git-hooks-pre-push-script-does-not-receive-input-via-stdin
> 
> Is this a bug in git?  Or am I just a shell noob?
> 
> Thanks in advance,
> 
> David

I assume that you have the right version of Git ? You can look into the source of Git, to be more exact the test suite: t/t5571-pre-push-hook.sh

When I play a little but with the script, (Replace the diff with test_cmp, add a line:

diff --git a/t/t5571-pre-push-hook.sh b/t/t5571-pre-push-hook.sh
index 6f9916a..01db2fb 100755
--- a/t/t5571-pre-push-hook.sh
+++ b/t/t5571-pre-push-hook.sh
@@ -42,6 +42,7 @@ export COMMIT2
 write_script "$HOOK" <<'EOF'
 echo "$1" >actual
 echo "$2" >>actual
+echo hello >>actual
 cat >>actual
 EOF
-------------
and finally run it like this:

debug=t verbose=t ./t5571-pre-push-hook.sh  2>&1 | less
I get something like this:

--- expected    2014-03-23 07:15:58.000000000 +0000
+++ actual      2014-03-23 07:15:58.000000000 +0000
@@ -1,3 +1,4 @@
 parent1
 repo1
+hello
 refs/heads/master 139b20d8e6c5b496de61f033f642d0e3dbff528d refs/heads/foreign d79ce1670bdcb76e6d1da2ae095e890ccb326ae9
not ok 4 - push with hook
#       
#               git push parent1 master:foreign &&
#               test_cmp expected actual
#       
--------------------------------------------
So the question is, if your push simply doesn't have anything to push,
because everything is up to date?
And in this case there is nothing on STDIN?

If the problem still exists, feel free to post a script how to reproduce it
here to the list, t5571 may be a source of inspiration.
David Cowden· Mar 23, 2014, 19:21 UTC · re: Torsten Bögershausen · lore

Re: Git-hooks pre-push script does not receive input on stdin

You are correct, it was a misunderstanding on my part. I have suggested a patch to clarify the pre-push documentation in a separate thread.

Thanks for pointing that out!
On Sun, Mar 23, 2014 at 12:19 AM, Torsten Bögershausen <tboegi@web.de> wrote:
Show 52 quoted lines
> On 2014-03-23 01.44, David Cowden wrote:
>> http://stackoverflow.com/questions/22585091/git-hooks-pre-push-script-does-not-receive-input-via-stdin
>>
>> Is this a bug in git?  Or am I just a shell noob?
>>
>> Thanks in advance,
>>
>> David
>
> I assume that you have the right version of Git ?
> You can look into the source of Git, to be more exact the test suite:
> t/t5571-pre-push-hook.sh
>
> When I play a little but with the script,
> (Replace the diff with test_cmp, add a line:
> diff --git a/t/t5571-pre-push-hook.sh b/t/t5571-pre-push-hook.sh
> index 6f9916a..01db2fb 100755
> --- a/t/t5571-pre-push-hook.sh
> +++ b/t/t5571-pre-push-hook.sh
> @@ -42,6 +42,7 @@ export COMMIT2
>  write_script "$HOOK" <<'EOF'
>  echo "$1" >actual
>  echo "$2" >>actual
> +echo hello >>actual
>  cat >>actual
>  EOF
> -------------
> and finally run it like this:
>
> debug=t verbose=t ./t5571-pre-push-hook.sh  2>&1 | less
> I get something like this:
>
> --- expected    2014-03-23 07:15:58.000000000 +0000
> +++ actual      2014-03-23 07:15:58.000000000 +0000
> @@ -1,3 +1,4 @@
>  parent1
>  repo1
> +hello
>  refs/heads/master 139b20d8e6c5b496de61f033f642d0e3dbff528d refs/heads/foreign d79ce1670bdcb76e6d1da2ae095e890ccb326ae9
> not ok 4 - push with hook
> #
> #               git push parent1 master:foreign &&
> #               test_cmp expected actual
> #
> --------------------------------------------
> So the question is, if your push simply doesn't have anything to push,
> because everything is up to date?
> And in this case there is nothing on STDIN?
>
> If the problem still exists, feel free to post a script how to reproduce it
> here to the list, t5571 may be a source of inspiration.
>

← back to recent threads