treewide: shorten temporary filename

[public-inbox.git] / script / lei

diff --git a/script/lei b/script/lei
index 58f0dbe92f90b36d243cfd46e1dc21a9aa31e96f..cb605e2eac1dc7bf9259b84e95dc506e5e8df4d9 100755 (executable)
--- a/script/lei
+++ b/script/lei
@@ -14,13 +14,15 @@ my $send_cmd = PublicInbox::CmdIPC4->can('send_cmd4') // do {
        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;
+       }
+};
 
-sub exec_cmd {
+my $exec_cmd = sub {
        my ($fds, $argc, @argv) = @_;
        my @old = (*STDIN{IO}, *STDOUT{IO}, *STDERR{IO});
        my @rdr;
@@ -29,7 +31,7 @@ sub exec_cmd {
                push @rdr, shift(@old), $tmpfh;
        }
        require POSIX; # WNOHANG
-       $SIG{CHLD} = \&sigchld;
+       $SIG{CHLD} = $sigchld;
        my $pid = fork // die "fork: $!";
        if ($pid == 0) {
                my %env = map { split(/=/, $_, 2) } splice(@argv, $argc);
@@ -38,9 +40,11 @@ sub exec_cmd {
                }
                %ENV = (%ENV, %env);
                exec(@argv);
-               die "exec: @argv: $!";
+               warn "exec: @argv: $!\n";
+               POSIX::_exit(1);
        }
-}
+       $pids{$pid} = 1;
+};
 
 if ($send_cmd && eval {
        my $path = do {
@@ -78,14 +82,6 @@ Falling back to (slow) one-shot mode
        while (my ($k, $v) = each %ENV) { $buf .= "\0$k=$v" }
        $buf .= "\0\0";
        my $n = $send_cmd->($sock, [0, 1, 2, fileno($dh)], $buf, MSG_EOR);
-       if (!$n && $!{ETOOMANYREFS} && eval { require BSD::Resource }) {
-               my $NOFILE = BSD::Resource::RLIMIT_NOFILE();
-               my ($s, $h) = BSD::Resource::getrlimit($NOFILE);
-               if ($s < $h && BSD::Resource::setrlimit($NOFILE, $h, $h)) {
-                       $n = $send_cmd->($sock, [0, 1, 2, fileno($dh)],
-                                       $buf, MSG_EOR);
-               }
-       }
        if (!$n) {
                die "sendmsg: $! (check RLIMIT_NOFILE)\n" if $!{ETOOMANYREFS};
                die "sendmsg: $!\n";
@@ -101,22 +97,24 @@ Falling back to (slow) one-shot mode
                        die "recvmsg: $!";
                }
                last if $buf eq '';
-               if ($buf =~ /\Ax_it ([0-9]+)\z/) {
+               if ($buf =~ /\Aexec (.+)\z/) {
+                       $exec_cmd->(\@fds, split(/\0/, $1));
+               } elsif ($buf eq '-WINCH') {
+                       kill($buf, $$); # for MUA
+               } elsif ($buf =~ /\Ax_it ([0-9]+)\z/) {
                        $x_it_code = $1 + 0;
                        last;
                } elsif ($buf =~ /\Achild_error ([0-9]+)\z/) {
                        $x_it_code = $1 + 0;
-               } elsif ($buf =~ /\Aexec (.+)\z/) {
-                       exec_cmd(\@fds, split(/\0/, $1));
                } else {
-                       sigchld();
+                       $sigchld->();
                        die $buf;
                }
        }
-       sigchld();
+       $sigchld->();
        if (my $sig = ($x_it_code & 127)) {
                kill $sig, $$;
-               sleep;
+               sleep(1) while 1;
        }
        exit($x_it_code >> 8);
 } else { # for systems lacking Socket::MsgHdr or Inline::C