From: Adrian Ratiu <adrian.ratiu@collabora•com>
To: git@vger•kernel.org, Jeff King <peff@peff•net>
Cc: Emily Shaffer <emilyshaffer@google•com>,
Junio C Hamano <gitster@pobox•com>,
Patrick Steinhardt <ps@pks•im>,
Josh Steadmon <steadmon@google•com>,
Kristoffer Haugsbakk <kristofferhaugsbakk@fastmail•com>,
Adrian Ratiu <adrian.ratiu@collabora•com>
Subject: [PATCH v8 00/12] Convert remaining hooks to hook.h
Date: Wed, 28 Jan 2026 23:39:15 +0200 [thread overview]
Message-ID: <20260128213927.3026875-1-adrian.ratiu@collabora.com> (raw)
In-Reply-To: <20250925125352.1728840-1-adrian.ratiu@collabora.com>
Hello everyone,
This series finishes the hook.[ch] conversion for the remaining hooks in
preparation for adding config-based hooks and enabling parallel hook
execution where possible (those will be separate series from this one).
v8 is just cleanups, function helpers and a bug fix, otherwise the new
design introduced in v7 is still the same. Especially I tried to clarify
the "muxer" sideband thread confusion with better explanations in comments
and commit messages.
Many thanks to all who contributed, it is appreciated!
This is based on the latest master branch, there are no conflicts with
next and seen, the code is available on GitHub and CI passes.
GitHub: https://github.com/10ne1/git/tree/dev/aratiu/hooks-conversion-v8
CI run: https://github.com/10ne1/git/actions/runs/21455314718
Changes in v8:
* Extensive parallel hook testing on Windows revealed a rare segfault triggered
by non-posix poll behavior, due to not checking the opts->feed_pipe() pointer
before calling it. It's fixed now by null-checking the pointer. (Adrian)
* Split t1800-hook test additions inte smaller independent tests, improved code
reuse & symmetry and split long lines (Junio)
* Improve header description of stdout_to_stderr (Patrick)
* Replaced .ungroup = options->jobs <= 1, with .ungroup = options->jobs == 1
because jobs can't be lower than 1, so the test is a bit clearer. (Adrian)
* Introduce a new "child_is_sending_output" run-command helper (Emily, Junio)
* Rename muxer thread and associated var to sideband_async for clarity
and reword commits to avoid "muxer" confusion (Emily, Adrian)
* Factored out common sideband async logic into helpers (Emily, Adrian)
* Fixed Suggested-by -> Helped-by tags and authorship (Adrian, Kristoffer)
* Commit message rewording improvements, tried to more clearly explain the
new design (Emily, Junio, Patrick, Kristoffer)
Range-diff v7 -> v8:
1: aff497683c < -: ---------- t1800: add hook output stream tests
-: ---------- > 1: 235337faa0 t1800: add hook output stream tests
2: a76027c3c6 ! 2: b880fe5210 run-command: add first helper for pp child states
@@ Metadata
Author: Adrian Ratiu <adrian.ratiu@collabora•com>
## Commit message ##
- run-command: add first helper for pp child states
+ run-command: add helper for pp child states
There is a recurring pattern of testing parallel process child states
and file descriptors to determine if a child is running, receiving any
input or if it's ready for cleanup.
- Name the pp_child structure and introduce a first helper to make these
- checks more readable. Next commits will add more helpers and checks.
+ Name the pp_child structure and introduce a helper to make the checks
+ more readable.
Suggested-by: Junio C Hamano <gitster@pobox•com>
Signed-off-by: Adrian Ratiu <adrian.ratiu@collabora•com>
3: a0969bbf3f = 3: 66c9f35c44 run-command: add stdin callback for parallelization
4: 43156cdfc4 = 4: 2f8b92923f hook: provide stdin via callback
5: 0cfd9ade6e = 5: 1e0425700d hook: convert 'post-rewrite' hook in sequencer.c to hook API
6: bb059a7508 ! 6: 957a8d4636 hook: allow separate std[out|err] streams
@@ Metadata
## Commit message ##
hook: allow separate std[out|err] streams
- The hook API assumed that all hooks merge stdout to stderr.
+ The hook API assumes that all hooks merge stdout to stderr.
This assumption is proven wrong by pre-push: some of its users
actually expect separate stdout and stderr streams and merging
@@ hook.h: struct run_hooks_opt
+ /**
+ * Send the hook's stdout to stderr.
++ *
++ * This is the default behavior for all hooks except pre-push,
++ * which has separate stdout and stderr streams for backwards
++ * compatibility reasons.
+ */
+ unsigned int stdout_to_stderr:1;
+
7: 7d14404316 = 7: 44005c911b transport: convert pre-push to hook API
8: c4a43509b8 = 8: 580197c9f7 reference-transaction: use hook API instead of run-command
9: 9336d6d752 ! 9: 097cd066e3 hook: add jobs option
@@ Commit message
This allows us to both pave the way for parallel hook execution
(that will be a follow-up patch series building upon this) and to
finish the API conversion of builtin/receive-pack.c, keeping the
- output muxer thread design as Peff suggested.
+ output async sideband thread ("muxer") design as Peff suggested.
- When .jobs == 1 nothing changes, the simple "copy_to_sideband" mux
+ When .jobs==1 nothing changes, the "copy_to_sideband" async thread
still outputs directly via sideband channel 2, keeping the current
- (mostly) real-time output characteristics, avoids unecessary poll
+ (mostly) real-time output characteristics, avoids unnecessary poll
delays or deadlock risks.
- When .jobs > 1, a more complex muxer is needed, to buffer the hook
+ When .jobs > 1, a more complex muxer is needed to buffer the hook
output and avoid interleaving. After working on this mux I quickly
realized I was re-implementing run-command with ungroup=0 so that
- ideas was dropped in favor of run-command which outputs to stderr.
- Then run-command's stderr just connects to the simple muxer input.
+ idea was dropped in favor of run-command which outputs to stderr.
- Maybe it's better to understand using ascii graphics:
+ In other words, run-command itself already can buffer/deinterleave
+ pp child outputs (ungroup=0), so we can just connect its stderr to
+ the sideband async task when jobs > 1.
+
+ Maybe it helps to illustrate how it works with ascii graphics:
[ Sequential (jobs = 1) ] [ Parallel (jobs > 1) ]
@@ Commit message
| | |
| | (dup2'd to pipe) |
| v |
- | +-------------------------+ |
- | | sideband "muxer" thread | |
- | +-------------------------+ |
+ | +-----------------------+ |
+ | | sideband async thread | |
+ | +-----------------------+ |
+-------------------------------------------------------------+
- When use_sideband == 0, the sideband muxer thread is missing, so
- the same architecture just outputs via the parent stderr stream.
+ When use_sideband == 0, the sideband async thread is missing, so
+ this same architecture just outputs via the parent stderr stream.
See the following commits for the hook API conversions doing this,
using pre-existing sideband thread logic from `copy_to_sideband`.
@@ hook.c: int run_hooks_opt(struct repository *r, const char *hook_name,
- .processes = 1,
- .ungroup = 1,
+ .processes = options->jobs,
-+ .ungroup = options->jobs <= 1,
++ .ungroup = options->jobs == 1,
.get_next_task = pick_next_hook,
.start_failure = notify_start_failure,
10: 1fac0e1894 ! 10: b7ffb59fb5 run-command: poll child stdin in addition to stdout
@@ Metadata
Author: Adrian Ratiu <adrian.ratiu@collabora•com>
## Commit message ##
- run-command: poll child stdin in addition to stdout
+ run-command: poll child input in addition to output
Child input feeding might hit the 100ms output poll timeout as a
side-effect of the ungroup=0 design when feeding multiple children
in parallel and buffering their outputs.
- This throttles the write throughtput as reported by Kristoffer.
+ This throttles the write throughput as reported by Kristoffer.
Peff also noted that the parent might block if the write pipe is full
and cause a deadlock if both parent + child wait for one another.
@@ Commit message
responsiveness for (sideband) output.
It's worth noting that in our current design, sequential execution
- is not affected by this because it still uses the ungroup=1 behavior.
+ is not affected by this because it still uses the ungroup=1 behavior,
+ so there are no run-command induced buffering delays since the child
+ sequentially outputs directly to the parent-inherited fds.
Reported-by: Kristoffer Haugsbakk <kristofferhaugsbakk@fastmail•com>
Suggested-by: Jeff King <peff@peff•net>
Signed-off-by: Adrian Ratiu <adrian.ratiu@collabora•com>
## run-command.c ##
+@@ run-command.c: static int child_is_receiving_input(const struct parallel_child *pp_child)
+ {
+ return child_is_working(pp_child) && pp_child->process.in > 0;
+ }
++static int child_is_sending_output(const struct parallel_child *pp_child)
++{
++ /*
++ * all pp children which buffer output through run_command via ungroup=0
++ * redirect stdout to stderr, so we just need to check process.err.
++ */
++ return child_is_working(pp_child) && pp_child->process.err > 0;
++}
+
+ struct parallel_processes {
+ size_t nr_processes;
@@ run-command.c: static void pp_init(struct parallel_processes *pp,
CALLOC_ARRAY(pp->children, n);
@@ run-command.c: static void pp_buffer_stdin(struct parallel_processes *pp,
- while (poll(pp->pfd, opts->processes, output_timeout) < 0) {
+ /* for each potential child slot, prepare two pollfd entries */
+ for (size_t i = 0; i < opts->processes; i++) {
-+ if (child_is_working(&pp->children[i]) &&
-+ pp->children[i].process.err > 0) {
++ if (child_is_sending_output(&pp->children[i])) {
+ pp->pfd[2*i].fd = pp->children[i].process.err;
+ pp->pfd[2*i].events = POLLIN | POLLHUP;
+ } else {
@@ run-command.c: static void pp_buffer_stdin(struct parallel_processes *pp,
for (size_t i = 0; i < opts->processes; i++) {
+ /* Handle input feeding (stdin) */
+ if (pp->pfd[2*i+1].revents & (POLLOUT | POLLHUP | POLLERR)) {
-+ int ret = opts->feed_pipe(pp->children[i].process.in,
-+ opts->data,
-+ pp->children[i].data);
-+ if (ret < 0)
-+ die_errno("feed_pipe");
-+ if (ret) {
++ if (opts->feed_pipe) {
++ int ret = opts->feed_pipe(pp->children[i].process.in,
++ opts->data,
++ pp->children[i].data);
++ if (ret < 0)
++ die_errno("feed_pipe");
++ if (ret) {
++ /* done feeding */
++ close(pp->children[i].process.in);
++ pp->children[i].process.in = 0;
++ }
++ } else {
++ /*
++ * No feed_pipe means there is nothing to do, so
++ * close the fd. Child input can be fed by other
++ * methods, such as opts->path_to_stdin which
++ * slurps a file via dup2, so clean up here.
++ */
+ close(pp->children[i].process.in);
+ pp->children[i].process.in = 0;
+ }
11: d9e48a9224 < -: ---------- receive-pack: convert update hooks to new API
-: ---------- > 11: b496510399 receive-pack: convert update hooks to new API
12: f8887c53e6 ! 12: fa24bf9032 receive-pack: convert receive hooks to hook API
@@
## Metadata ##
-Author: Adrian Ratiu <adrian.ratiu@collabora•com>
+Author: Emily Shaffer <nasamuffin@google•com>
## Commit message ##
receive-pack: convert receive hooks to hook API
@@ Commit message
signals, manage custom struct child_process, call find_hook(),
prepares for specifying hooks via configs, etc.).
- See the previous four commits for a more details explanation how
- this all works.
+ See the previous three commits for a more in-depth explanation of
+ how this all works.
- Suggested-by: Jeff King <peff@peff•net>
Signed-off-by: Emily Shaffer <emilyshaffer@google•com>
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail•com>
+ Helped-by: Jeff King <peff@peff•net>
Signed-off-by: Adrian Ratiu <adrian.ratiu@collabora•com>
## builtin/receive-pack.c ##
@@ builtin/receive-pack.c: static int run_receive_hook(struct command *commands,
+ struct run_hooks_opt opt = RUN_HOOKS_OPT_INIT;
+ struct command *iter = commands;
+ struct receive_hook_feed_state feed_state;
-+ struct async muxer;
-+ int ret;
++ struct async sideband_async;
++ int sideband_async_started = 0;
+ int saved_stderr = -1;
-+ int muxer_started = 0;
++ int ret;
- strbuf_init(&state.buf, 0);
- state.cmd = commands;
@@ builtin/receive-pack.c: static int run_receive_hook(struct command *commands,
+
+ prepare_push_cert_sha1(&opt);
+
-+ if (use_sideband) {
-+ memset(&muxer, 0, sizeof(muxer));
-+ muxer.proc = copy_to_sideband;
-+ muxer.in = -1;
-+ if (!start_async(&muxer)) {
-+ muxer_started = 1;
-+ saved_stderr = dup(STDERR_FILENO);
-+ if (saved_stderr >= 0)
-+ dup2(muxer.in, STDERR_FILENO);
-+ close(muxer.in);
-+ }
-+ }
++ prepare_sideband_async(&sideband_async, &saved_stderr, &sideband_async_started);
+
+ /* set up stdin callback */
+ feed_state.cmd = commands;
@@ builtin/receive-pack.c: static int run_receive_hook(struct command *commands,
+ ret = run_hooks_opt(the_repository, hook_name, &opt);
+
+ strbuf_release(&feed_state.buf);
-+ if (saved_stderr >= 0) {
-+ dup2(saved_stderr, STDERR_FILENO);
-+ close(saved_stderr);
-+ }
-+ if (muxer_started)
-+ finish_async(&muxer);
++ finish_sideband_async(&sideband_async, saved_stderr, sideband_async_started);
+
+ return ret;
}
Adrian Ratiu (6):
t1800: add hook output stream tests
run-command: add helper for pp child states
hook: allow separate std[out|err] streams
reference-transaction: use hook API instead of run-command
hook: add jobs option
run-command: poll child input in addition to output
Emily Shaffer (6):
run-command: add stdin callback for parallelization
hook: provide stdin via callback
hook: convert 'post-rewrite' hook in sequencer.c to hook API
transport: convert pre-push to hook API
receive-pack: convert update hooks to new API
receive-pack: convert receive hooks to hook API
builtin/receive-pack.c | 283 ++++++++++++++++++------------------
hook.c | 32 +++-
hook.h | 57 ++++++++
refs.c | 102 ++++++-------
run-command.c | 174 ++++++++++++++++++----
run-command.h | 21 +++
sequencer.c | 42 +++---
t/helper/test-run-command.c | 52 ++++++-
t/t0061-run-command.sh | 31 ++++
t/t1800-hook.sh | 137 +++++++++++++++++
transport.c | 95 ++++++------
11 files changed, 743 insertions(+), 283 deletions(-)
--
2.52.0.732.gb351b5166d.dirty
next prev parent reply other threads:[~2026-01-28 21:40 UTC|newest]
Thread overview: 187+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-09-25 12:53 [PATCH 00/10] Convert remaining hooks to hook.h Adrian Ratiu
2025-09-25 12:53 ` [PATCH 01/10] run-command: add stdin callback for parallelization Adrian Ratiu
2025-10-02 6:34 ` Patrick Steinhardt
2025-10-02 15:46 ` Junio C Hamano
2025-10-06 13:01 ` Adrian Ratiu
2025-10-06 12:59 ` Adrian Ratiu
2025-10-14 17:35 ` Adrian Ratiu
2025-09-25 12:53 ` [PATCH 02/10] hook: provide stdin via callback Adrian Ratiu
2025-09-25 20:05 ` Junio C Hamano
2025-09-26 12:03 ` Adrian Ratiu
2025-10-10 19:57 ` Emily Shaffer
2025-10-13 14:47 ` Adrian Ratiu
2025-09-25 12:53 ` [PATCH 03/10] hook: convert 'post-rewrite' hook in sequencer.c to hook.h Adrian Ratiu
2025-09-25 20:15 ` Junio C Hamano
2025-09-26 12:29 ` Adrian Ratiu
2025-09-26 14:12 ` Phillip Wood
2025-09-26 15:53 ` Adrian Ratiu
2025-09-29 10:11 ` Phillip Wood
2025-09-26 17:52 ` Junio C Hamano
2025-09-29 7:33 ` Adrian Ratiu
2025-10-02 6:34 ` Patrick Steinhardt
2025-10-08 7:04 ` Adrian Ratiu
2025-09-25 12:53 ` [PATCH 04/10] transport: convert pre-push hook " Adrian Ratiu
2025-09-25 18:58 ` D. Ben Knoble
2025-09-26 13:02 ` Adrian Ratiu
2025-09-26 14:11 ` Phillip Wood
2025-09-29 11:33 ` Adrian Ratiu
2025-09-25 12:53 ` [PATCH 05/10] reference-transaction: use hook.h to run hooks Adrian Ratiu
2025-09-25 21:45 ` Junio C Hamano
2025-09-26 13:03 ` Adrian Ratiu
2025-10-02 6:34 ` Patrick Steinhardt
2025-10-08 12:26 ` Adrian Ratiu
2025-09-25 12:53 ` [PATCH 06/10] run-command: allow capturing of collated output Adrian Ratiu
2025-09-25 21:52 ` Junio C Hamano
2025-09-26 14:14 ` Adrian Ratiu
2025-09-25 12:53 ` [PATCH 07/10] hooks: allow callers to capture output Adrian Ratiu
2025-09-25 12:53 ` [PATCH 08/10] receive-pack: convert 'update' hook to hook.h Adrian Ratiu
2025-09-25 21:53 ` Junio C Hamano
2025-10-10 19:57 ` Emily Shaffer
2025-10-17 8:27 ` Adrian Ratiu
2025-09-25 12:53 ` [PATCH 09/10] post-update: use hook.h library Adrian Ratiu
2025-09-25 18:02 ` [PATCH 10/10] receive-pack: convert receive hooks to hook.h Adrian Ratiu
2025-10-10 19:57 ` [PATCH 00/10] Convert remaining " Emily Shaffer
2025-10-17 14:15 ` [PATCH v2 " Adrian Ratiu
2025-10-17 14:15 ` [PATCH v2 01/10] run-command: add stdin callback for parallelization Adrian Ratiu
2025-10-21 7:40 ` Patrick Steinhardt
2025-10-17 14:15 ` [PATCH v2 02/10] hook: provide stdin via callback Adrian Ratiu
2025-10-21 7:41 ` Patrick Steinhardt
2025-10-21 7:41 ` Patrick Steinhardt
2025-10-21 14:44 ` Adrian Ratiu
2025-10-17 14:15 ` [PATCH v2 03/10] hook: convert 'post-rewrite' hook in sequencer.c to hook API Adrian Ratiu
2025-10-21 7:41 ` Patrick Steinhardt
2025-10-21 15:44 ` Adrian Ratiu
2025-10-17 14:15 ` [PATCH v2 04/10] transport: convert pre-push " Adrian Ratiu
2025-10-21 7:41 ` Patrick Steinhardt
2025-10-21 16:04 ` Adrian Ratiu
2025-10-17 14:15 ` [PATCH v2 05/10] reference-transaction: use hook API instead of run-command Adrian Ratiu
2025-10-17 14:15 ` [PATCH v2 06/10] hook: allow overriding the ungroup option Adrian Ratiu
2025-10-17 14:15 ` [PATCH v2 07/10] run-command: allow capturing of collated output Adrian Ratiu
2025-10-21 7:41 ` Patrick Steinhardt
2025-10-21 16:25 ` Adrian Ratiu
2025-10-17 14:15 ` [PATCH v2 08/10] hooks: allow callers to capture output Adrian Ratiu
2025-10-17 14:15 ` [PATCH v2 09/10] receive-pack: convert update hooks to new API Adrian Ratiu
2025-10-28 18:39 ` Kristoffer Haugsbakk
2025-10-17 14:15 ` [PATCH v2 10/10] receive-pack: convert receive hooks to hook API Adrian Ratiu
2025-10-21 7:41 ` Patrick Steinhardt
2025-10-28 18:42 ` Kristoffer Haugsbakk
2025-10-29 13:46 ` Adrian Ratiu
2025-10-29 13:50 ` Kristoffer Haugsbakk
2025-11-15 19:48 ` Junio C Hamano
2025-11-17 16:51 ` Adrian Ratiu
2025-10-21 7:40 ` [PATCH v2 00/10] Convert remaining hooks to hook.h Patrick Steinhardt
2025-10-21 16:34 ` Adrian Ratiu
2025-11-24 17:20 ` [PATCH v3 " Adrian Ratiu
2025-11-24 17:20 ` [PATCH v3 01/10] run-command: add stdin callback for parallelization Adrian Ratiu
2025-11-25 23:15 ` Junio C Hamano
2025-11-27 12:00 ` Adrian Ratiu
2025-11-24 17:20 ` [PATCH v3 02/10] hook: provide stdin via callback Adrian Ratiu
2025-11-29 13:03 ` Adrian Ratiu
2025-11-29 22:21 ` Junio C Hamano
2025-12-01 13:26 ` Adrian Ratiu
2025-11-24 17:20 ` [PATCH v3 03/10] hook: convert 'post-rewrite' hook in sequencer.c to hook API Adrian Ratiu
2025-11-24 17:20 ` [PATCH v3 04/10] transport: convert pre-push " Adrian Ratiu
2025-11-24 22:55 ` Junio C Hamano
2025-11-27 14:24 ` Adrian Ratiu
2025-11-24 17:20 ` [PATCH v3 05/10] reference-transaction: use hook API instead of run-command Adrian Ratiu
2025-11-24 17:20 ` [PATCH v3 06/10] hook: allow overriding the ungroup option Adrian Ratiu
2025-11-24 17:20 ` [PATCH v3 07/10] run-command: allow capturing of collated output Adrian Ratiu
2025-11-24 17:20 ` [PATCH v3 08/10] hooks: allow callers to capture output Adrian Ratiu
2025-11-24 17:20 ` [PATCH v3 09/10] receive-pack: convert update hooks to new API Adrian Ratiu
2025-11-24 17:20 ` [PATCH v3 10/10] receive-pack: convert receive hooks to hook API Adrian Ratiu
2025-12-04 14:15 ` [PATCH v4 00/11] Convert remaining hooks to hook.h Adrian Ratiu
2025-12-04 14:15 ` [PATCH v4 01/11] run-command: add first helper for pp child states Adrian Ratiu
2025-12-04 14:15 ` [PATCH v4 02/11] run-command: add stdin callback for parallelization Adrian Ratiu
2025-12-04 14:15 ` [PATCH v4 03/11] hook: provide stdin via callback Adrian Ratiu
2025-12-16 8:08 ` Patrick Steinhardt
2025-12-04 14:15 ` [PATCH v4 04/11] hook: convert 'post-rewrite' hook in sequencer.c to hook API Adrian Ratiu
2025-12-04 14:15 ` [PATCH v4 05/11] transport: convert pre-push " Adrian Ratiu
2025-12-16 8:08 ` Patrick Steinhardt
2025-12-16 9:09 ` Adrian Ratiu
2025-12-16 9:30 ` Patrick Steinhardt
2025-12-17 23:07 ` Junio C Hamano
2025-12-04 14:15 ` [PATCH v4 06/11] reference-transaction: use hook API instead of run-command Adrian Ratiu
2025-12-04 14:15 ` [PATCH v4 07/11] hook: allow overriding the ungroup option Adrian Ratiu
2025-12-04 14:15 ` [PATCH v4 08/11] run-command: allow capturing of collated output Adrian Ratiu
2025-12-04 14:15 ` [PATCH v4 09/11] hooks: allow callers to capture output Adrian Ratiu
2025-12-04 14:15 ` [PATCH v4 10/11] receive-pack: convert update hooks to new API Adrian Ratiu
2025-12-16 8:08 ` Patrick Steinhardt
2025-12-16 9:22 ` Adrian Ratiu
2025-12-04 14:15 ` [PATCH v4 11/11] receive-pack: convert receive hooks to hook API Adrian Ratiu
2025-12-18 17:11 ` [PATCH v5 00/11] Convert remaining hooks to hook.h Adrian Ratiu
2025-12-18 17:11 ` [PATCH v5 01/11] run-command: add first helper for pp child states Adrian Ratiu
2025-12-18 17:11 ` [PATCH v5 02/11] run-command: add stdin callback for parallelization Adrian Ratiu
2025-12-18 17:11 ` [PATCH v5 03/11] hook: provide stdin via callback Adrian Ratiu
2025-12-18 17:11 ` [PATCH v5 04/11] hook: convert 'post-rewrite' hook in sequencer.c to hook API Adrian Ratiu
2025-12-18 17:11 ` [PATCH v5 05/11] transport: convert pre-push " Adrian Ratiu
2025-12-18 17:11 ` [PATCH v5 06/11] reference-transaction: use hook API instead of run-command Adrian Ratiu
2025-12-18 17:11 ` [PATCH v5 07/11] hook: allow overriding the ungroup option Adrian Ratiu
2025-12-18 17:11 ` [PATCH v5 08/11] run-command: allow capturing of collated output Adrian Ratiu
2025-12-18 17:11 ` [PATCH v5 09/11] hooks: allow callers to capture output Adrian Ratiu
2025-12-18 17:11 ` [PATCH v5 10/11] receive-pack: convert update hooks to new API Adrian Ratiu
2025-12-18 17:11 ` [PATCH v5 11/11] receive-pack: convert receive hooks to hook API Adrian Ratiu
2025-12-19 12:38 ` Patrick Steinhardt
2025-12-20 10:40 ` Adrian Ratiu
2025-12-26 12:23 ` [PATCH v6 00/11] Convert remaining hooks to hook.h Adrian Ratiu
2025-12-26 12:23 ` [PATCH v6 01/11] run-command: add first helper for pp child states Adrian Ratiu
2025-12-26 12:23 ` [PATCH v6 02/11] run-command: add stdin callback for parallelization Adrian Ratiu
2025-12-26 12:23 ` [PATCH v6 03/11] hook: provide stdin via callback Adrian Ratiu
2025-12-26 12:23 ` [PATCH v6 04/11] hook: convert 'post-rewrite' hook in sequencer.c to hook API Adrian Ratiu
2025-12-26 12:23 ` [PATCH v6 05/11] transport: convert pre-push " Adrian Ratiu
2025-12-26 12:23 ` [PATCH v6 06/11] reference-transaction: use hook API instead of run-command Adrian Ratiu
2026-01-18 12:23 ` SZEDER Gábor
2026-01-18 18:30 ` Adrian Ratiu
2025-12-26 12:23 ` [PATCH v6 07/11] hook: allow overriding the ungroup option Adrian Ratiu
2025-12-26 12:23 ` [PATCH v6 08/11] run-command: allow capturing of collated output Adrian Ratiu
2025-12-26 12:23 ` [PATCH v6 09/11] hooks: allow callers to capture output Adrian Ratiu
2025-12-26 12:23 ` [PATCH v6 10/11] receive-pack: convert update hooks to new API Adrian Ratiu
2025-12-26 12:23 ` [PATCH v6 11/11] receive-pack: convert receive hooks to hook API Adrian Ratiu
2025-12-28 11:32 ` [PATCH v6 00/11] Convert remaining hooks to hook.h Junio C Hamano
2026-01-05 10:52 ` Adrian Ratiu
2026-01-05 12:13 ` Patrick Steinhardt
2026-01-21 21:54 ` [PATCH v7 00/12] " Adrian Ratiu
2026-01-21 21:54 ` [PATCH v7 01/12] t1800: add hook output stream tests Adrian Ratiu
2026-01-21 22:16 ` Junio C Hamano
2026-01-22 9:19 ` Adrian Ratiu
2026-01-21 21:54 ` [PATCH v7 02/12] run-command: add first helper for pp child states Adrian Ratiu
2026-01-21 23:01 ` Junio C Hamano
2026-01-22 9:21 ` Adrian Ratiu
2026-01-21 21:54 ` [PATCH v7 03/12] run-command: add stdin callback for parallelization Adrian Ratiu
2026-01-21 21:54 ` [PATCH v7 04/12] hook: provide stdin via callback Adrian Ratiu
2026-01-21 21:54 ` [PATCH v7 05/12] hook: convert 'post-rewrite' hook in sequencer.c to hook API Adrian Ratiu
2026-01-21 21:54 ` [PATCH v7 06/12] hook: allow separate std[out|err] streams Adrian Ratiu
2026-01-23 7:19 ` Patrick Steinhardt
2026-01-23 7:47 ` Adrian Ratiu
2026-01-21 21:54 ` [PATCH v7 07/12] transport: convert pre-push to hook API Adrian Ratiu
2026-01-21 21:54 ` [PATCH v7 08/12] reference-transaction: use hook API instead of run-command Adrian Ratiu
2026-01-21 21:54 ` [PATCH v7 09/12] hook: add jobs option Adrian Ratiu
2026-01-21 21:54 ` [PATCH v7 10/12] run-command: poll child stdin in addition to stdout Adrian Ratiu
2026-01-21 22:04 ` Kristoffer Haugsbakk
2026-01-22 9:57 ` Adrian Ratiu
2026-01-21 23:11 ` Junio C Hamano
2026-01-22 10:58 ` Adrian Ratiu
2026-01-22 17:20 ` Junio C Hamano
2026-01-26 23:20 ` Emily Shaffer
2026-01-27 0:11 ` Junio C Hamano
2026-01-27 10:10 ` Adrian Ratiu
2026-01-21 21:54 ` [PATCH v7 11/12] receive-pack: convert update hooks to new API Adrian Ratiu
2026-01-21 22:14 ` Kristoffer Haugsbakk
2026-01-22 9:26 ` Adrian Ratiu
2026-01-27 0:12 ` Emily Shaffer
2026-01-27 13:05 ` Adrian Ratiu
2026-01-21 21:54 ` [PATCH v7 12/12] receive-pack: convert receive hooks to hook API Adrian Ratiu
2026-01-28 21:39 ` Adrian Ratiu [this message]
2026-01-28 21:39 ` [PATCH v8 01/12] t1800: add hook output stream tests Adrian Ratiu
2026-01-28 21:39 ` [PATCH v8 02/12] run-command: add helper for pp child states Adrian Ratiu
2026-01-28 21:39 ` [PATCH v8 03/12] run-command: add stdin callback for parallelization Adrian Ratiu
2026-01-28 21:39 ` [PATCH v8 04/12] hook: provide stdin via callback Adrian Ratiu
2026-01-28 21:39 ` [PATCH v8 05/12] hook: convert 'post-rewrite' hook in sequencer.c to hook API Adrian Ratiu
2026-01-28 21:39 ` [PATCH v8 06/12] hook: allow separate std[out|err] streams Adrian Ratiu
2026-02-02 3:17 ` Chris Darroch
2026-02-02 16:32 ` Junio C Hamano
2026-01-28 21:39 ` [PATCH v8 07/12] transport: convert pre-push to hook API Adrian Ratiu
2026-01-28 21:39 ` [PATCH v8 08/12] reference-transaction: use hook API instead of run-command Adrian Ratiu
2026-01-28 21:39 ` [PATCH v8 09/12] hook: add jobs option Adrian Ratiu
2026-01-28 21:39 ` [PATCH v8 10/12] run-command: poll child input in addition to output Adrian Ratiu
2026-01-28 21:39 ` [PATCH v8 11/12] receive-pack: convert update hooks to new API Adrian Ratiu
2026-01-28 21:39 ` [PATCH v8 12/12] receive-pack: convert receive hooks to hook API Adrian Ratiu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260128213927.3026875-1-adrian.ratiu@collabora.com \
--to=adrian.ratiu@collabora$(echo .)com \
--cc=emilyshaffer@google$(echo .)com \
--cc=git@vger$(echo .)kernel.org \
--cc=gitster@pobox$(echo .)com \
--cc=kristofferhaugsbakk@fastmail$(echo .)com \
--cc=peff@peff$(echo .)net \
--cc=ps@pks$(echo .)im \
--cc=steadmon@google$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox