We only spawn one process to be reaped at the moment. tests
will run the contents of script/* in the same process if
possible, so any test scripts which spawn -httpd or other
read-only can cause us to stall with waitpid(-1, ...)
PublicInbox::Spawn->can('send_cmd4');
};
PublicInbox::Spawn->can('send_cmd4');
};
-sub sigchld {
- my ($sig) = @_;
- my $flags = $sig ? POSIX::WNOHANG() : 0;
- while (waitpid(-1, $flags) > 0) {}
-}
+my %pids;
+my $sigchld = sub {
+ my $flags = scalar(@_) ? POSIX::WNOHANG() : 0;
+ for my $pid (keys %pids) {
+ delete($pids{$pid}) if waitpid($pid, $flags) == $pid;
+ }
+};
my ($fds, $argc, @argv) = @_;
my @old = (*STDIN{IO}, *STDOUT{IO}, *STDERR{IO});
my @rdr;
my ($fds, $argc, @argv) = @_;
my @old = (*STDIN{IO}, *STDOUT{IO}, *STDERR{IO});
my @rdr;
push @rdr, shift(@old), $tmpfh;
}
require POSIX; # WNOHANG
push @rdr, shift(@old), $tmpfh;
}
require POSIX; # WNOHANG
- $SIG{CHLD} = \&sigchld;
my $pid = fork // die "fork: $!";
if ($pid == 0) {
my %env = map { split(/=/, $_, 2) } splice(@argv, $argc);
my $pid = fork // die "fork: $!";
if ($pid == 0) {
my %env = map { split(/=/, $_, 2) } splice(@argv, $argc);
}
%ENV = (%ENV, %env);
exec(@argv);
}
%ENV = (%ENV, %env);
exec(@argv);
+ warn "exec: @argv: $!\n";
+ POSIX::_exit(1);
if ($send_cmd && eval {
my $path = do {
if ($send_cmd && eval {
my $path = do {
} elsif ($buf =~ /\Achild_error ([0-9]+)\z/) {
$x_it_code = $1 + 0;
} elsif ($buf =~ /\Aexec (.+)\z/) {
} elsif ($buf =~ /\Achild_error ([0-9]+)\z/) {
$x_it_code = $1 + 0;
} elsif ($buf =~ /\Aexec (.+)\z/) {
- exec_cmd(\@fds, split(/\0/, $1));
+ $exec_cmd->(\@fds, split(/\0/, $1));
if (my $sig = ($x_it_code & 127)) {
kill $sig, $$;
sleep(1) while 1;
if (my $sig = ($x_it_code & 127)) {
kill $sig, $$;
sleep(1) while 1;