[PATCH] hook: make stdout_to_stderr optional
- From
Adrian Ratiu <adrian.ratiu@collabora.com>
- Date
- Jan 13, 2026, 11:56 UTC
- Message-ID
- <20260113115633.230479-1-adrian.ratiu@collabora.com>
The last batch of hooks converted to the hook.[ch] API introduced a regression because pick_next_hook() always sets stdout_to_stderr for its child processes.
Pre-push is the only hook API user which requires stdout_to_stderr to be 0, so it can be argued that pre-push needs fixing, however this will likely break many pre-push hooks, so it's better to allow it to be 0, i.e. to match the previous behavior.
We can introduce an extension for the breaking change of all hooks sending stdout to stderr, however this just fixes the regression.
Reported-by: Chris Darroch <chrisd@apache.org> Suggested-by: brian m. carlson <sandals@crustytoothpaste.net> Signed-off-by: Adrian Ratiu <adrian.ratiu@collabora.com> --- This is based on the latest master branch. Pushed to GitHub: https://github.com/10ne1/git/tree/dev/aratiu/make-hook-stdout_to_stderr-optional-v1 Succesful CI run: https://github.com/10ne1/git/actions/runs/20954859587 --- hook.c | 2 +- hook.h | 6 ++++++ transport.c | 1 + 3 files changed, 8 insertions(+), 1 deletion(-)
diff --git a/hook.c b/hook.c index 35211e5ed7..ebd9d9e26e 100644 --- a/hook.c +++ b/hook.c @@ -81,7 +81,7 @@ static int pick_next_hook(struct child_process *cp, cp->in = -1; } - cp->stdout_to_stderr = 1; + cp->stdout_to_stderr = hook_cb->options->stdout_to_stderr; cp->trace2_hook_name = hook_cb->hook_name; cp->dir = hook_cb->options->dir; diff --git a/hook.h b/hook.h index ae502178b9..2488db7133 100644 --- a/hook.h +++ b/hook.h @@ -39,6 +39,11 @@ struct run_hooks_opt */ unsigned int ungroup:1; + /** + * Send the hook's stdout to stderr. + */ + unsigned int stdout_to_stderr:1; + /** * Path to file which should be piped to stdin for each hook. */ @@ -93,6 +98,7 @@ struct run_hooks_opt #define RUN_HOOKS_OPT_INIT { \ .env = STRVEC_INIT, \ .args = STRVEC_INIT, \ + .stdout_to_stderr = 1, \ } struct hook_cb_data { diff --git a/transport.c b/transport.c index 6d0f02be5d..8f0e5987ab 100644 --- a/transport.c +++ b/transport.c @@ -1372,6 +1372,7 @@ static int run_pre_push_hook(struct transport *transport, opt.feed_pipe = pre_push_hook_feed_stdin; opt.feed_pipe_cb_data = &data; + opt.stdout_to_stderr = 0; ret = run_hooks_opt(the_repository, "pre-push", &opt);
-- 2.52.0