Thread (29 messages) 29 messages, 6 authors, 2026-05-22

Re: [PATCH 4/9] run-command: add support for timeout in command finisher

From: Oswald Buddenhagen <hidden>
Date: 2026-05-21 08:39:28

On Thu, May 21, 2026 at 09:21:51AM +0200, Johannes Sixt wrote:
Please, do not add this infrastructure to core Git, and instead fix the
communication protocol.
there is nothing to fix here. proper error handling including timeout 
handling should just be part of every protocol handler, and in the case 
of child processes, forcible termination is part of that.

one can ignore the issue, in which case termination is left to the user 
by ctrl-c'ing the whole process group. this isn't very user-friendly, 
because it doesn't report the problem, and it may leave hung processes 
behind. it is also extremely bad if keeping the parent process alive is 
a lot more important than the child process, but this doesn't appear to 
apply to the particular use case.

adding a proxy doesn't fix the problem, it just adds another point of 
failure.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help