lei q: fix MUA spawn after reading query from stdin

[public-inbox.git] / script / lei

diff --git a/script/lei b/script/lei
index b7f21f1410fab4237054542cd2cdfbec72b0f8c1..56e9d299593b95c907aaf2a8482ce862a91d5aba 100755 (executable)
--- a/script/lei
+++ b/script/lei
@@ -21,29 +21,42 @@ my $sigchld = sub {
                delete($pids{$pid}) if waitpid($pid, $flags) == $pid;
        }
 };
-
+my @parent;
 my $exec_cmd = sub {
        my ($fds, $argc, @argv) = @_;
+       my $parent = $$;
+       require POSIX;
        my @old = (*STDIN{IO}, *STDOUT{IO}, *STDERR{IO});
        my @rdr;
        for my $fd (@$fds) {
-               open(my $tmpfh, '+<&=', $fd) or die "open +<&=$fd: $!";
-               push @rdr, shift(@old), $tmpfh;
+               open(my $newfh, '+<&=', $fd) or die "open +<&=$fd: $!";
+               push @rdr, shift(@old), $newfh;
        }
-       require POSIX; # WNOHANG
-       $SIG{CHLD} = $sigchld;
-       my $pid = fork // die "fork: $!";
-       if ($pid == 0) {
-               my %env = map { split(/=/, $_, 2) } splice(@argv, $argc);
-               while (my ($old_io, $tmpfh) = splice(@rdr, 0, 2)) {
-                       open $old_io, '+<&', $tmpfh or die "open +<&=: $!";
+       my $do_exec = sub {
+               while (my ($io, $newfh) = splice(@rdr, 0, 2)) {
+                       open $io, '+<&', $newfh or die "open +<&=: $!";
                }
-               %ENV = (%ENV, %env);
+               my %env = map { split(/=/, $_, 2) } splice(@argv, $argc);
+               @ENV{keys %env} = values %env;
                exec(@argv);
                warn "exec: @argv: $!\n";
                POSIX::_exit(1);
+       };
+       $SIG{CHLD} = $sigchld;
+       my $pid = fork // die "fork: $!";
+       if ($pid == 0) {
+               $do_exec->() if $fds->[1]; # git-credential, pager
+
+               # parent backgrounds on MUA
+               POSIX::setsid() > 0 or die "setsid: $!";
+               @parent = ($parent);
+               return; # continue $recv_cmd in background
+       }
+       if ($fds->[1]) {
+               $pids{$pid} = undef;
+       } else {
+               $do_exec->(); # MUA reuses stdout
        }
-       $pids{$pid} = 1;
 };
 
 if ($send_cmd && eval {
@@ -82,14 +95,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";
@@ -105,13 +110,15 @@ 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, @parent); # 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->();
                        die $buf;