diff options
author | Eric Wong <e@80x24.org> | 2021-10-14 13:16:07 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2021-10-15 02:23:26 +0000 |
commit | 4b5a1b5787edee2a3b6cc10a3ccc5721f1414268 (patch) | |
tree | 5439effe167100e8c2865a92ddc4df4b2bf6c0df /lib/PublicInbox/LeiToMail.pm | |
parent | f64fdbc7cc19a34f471abaeada9195597bf7a282 (diff) | |
download | public-inbox-4b5a1b5787edee2a3b6cc10a3ccc5721f1414268.tar.gz |
By relying more on pgroups for remaining remaining processes, this lets us pause all curl+tail subprocesses with a single kill(2) to avoid cluttering stderr. We won't bother pausing the pigz/gzip/bzip2/xz compressor process not cat-file processes, though, since those don't write to the terminal and they idle soon after the workers react to SIGSTOP. AutoReap is hoisted out from TestCommon.pm. CLONE_SKIP is gone since we won't be using Perl threads any time soon (they're discouraged by the maintainers of Perl).
Diffstat (limited to 'lib/PublicInbox/LeiToMail.pm')
-rw-r--r-- | lib/PublicInbox/LeiToMail.pm | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/PublicInbox/LeiToMail.pm b/lib/PublicInbox/LeiToMail.pm index 5a220ba3..9c748dea 100644 --- a/lib/PublicInbox/LeiToMail.pm +++ b/lib/PublicInbox/LeiToMail.pm @@ -157,7 +157,7 @@ sub _post_augment_mbox { # open a compressor process from top-level process my $zsfx = $self->{zsfx} or return; my $cmd = PublicInbox::MboxReader::zsfx2cmd($zsfx, undef, $lei); my ($r, $w) = @{delete $lei->{zpipe}}; - my $rdr = { 0 => $r, 1 => $lei->{1}, 2 => $lei->{2} }; + my $rdr = { 0 => $r, 1 => $lei->{1}, 2 => $lei->{2}, pgid => 0 }; my $pid = spawn($cmd, undef, $rdr); my $pp = gensym; my $dup = bless { "pid.$pid" => $cmd }, ref($lei); |