Re: bug report: pre-push hook

From: Taylor Blau <me@ttaylorr.com>
Date: Mar 24, 2022, 21:29 UTC
Message-ID: <Yjziu8nU/SZ7SyH0@nand.local>
In-Reply-To: <CAJoAoZmo4bJgGdYQeM4XtRJHOpm4z7Gvg=qpUc4xa7Ap=v++UA@mail.gmail.com>

On Thu, Mar 24, 2022 at 11:30:16AM -0700, Emily Shaffer wrote:

Show 6 quoted lines

> > What happened instead? (Actual behavior)
> > Tests seem to execute on the background, but I do not see output
>
> I wonder - are you printing your test output to stdout, or stderr?
> stderr of the hook is printed to stdout of the Git process, so that
> may be why.

I didn't think that we redirected the pre-push (or any) hook's stderr to stdout in the parent process. I wrote a small pre-push hook which is just:

    $ cat .git/hooks/pre-push
    #!/bin/sh

    echo >&2 "hi!"

and then:

    $ git.compile push ttaylorr --dry-run 2>foo
    $ grep hi foo
    hi!

But I might be holding it wrong, since I am not a frequent user of hooks.

Thanks, Taylor