diff options
author | Eric Wong <e@80x24.org> | 2016-06-18 10:53:32 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2016-06-18 21:48:25 +0000 |
commit | e748f75979046724ead380bd00eadc677bdc07e9 (patch) | |
tree | fbcd16686324cc689a1cd1b0de642bc9a9881601 /t | |
parent | 1701875544e31d77f65cf467a35c8dd2bcebf8fa (diff) | |
download | public-inbox-e748f75979046724ead380bd00eadc677bdc07e9.tar.gz |
While we only want to stop our daemons and gracefully destroy subprocesses, it is common for 'Ctrl-C' from a terminal to kill the entire pgroup. Killing an entire pgroup nukes subprocesses like git-upload-pack breaks graceful shutdown on long clones. Make a best effort to ensure git-upload-pack processes are not broken when somebody signals an entire process group. Followup-to: commit 37bf2db81bbbe114d7fc5a00e30d3d5a6fa74de5 ("doc: systemd examples should only kill one process")
Diffstat (limited to 't')
-rw-r--r-- | t/spawn.t | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -87,7 +87,7 @@ use PublicInbox::Spawn qw(which spawn popen_rd); is(kill(0, $pid), 1, 'child process is running'); ok(!defined(sysread($fh, my $buf, 1)) && $!{EAGAIN}, 'sysread returned quickly with EAGAIN'); - is(kill(15, $pid), 1, 'child process killed early'); + is(kill(9, $pid), 1, 'child process killed early'); is(waitpid($pid, 0), $pid, 'child process reapable'); isnt($?, 0, '$? set properly: '.$?); } |