]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/Gcf2Client.pm
rename {pi_config} fields to {pi_cfg}
[public-inbox.git] / lib / PublicInbox / Gcf2Client.pm
index a048cd1a6e26bcbe799647115bc18c38f46c1d73..ab486de5895ad9e5190d3438b7adb029414dfa90 100644 (file)
@@ -1,42 +1,69 @@
 # Copyright (C) 2020 all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 
-# connects public-inbox processes to public-inbox-gcf2(1)
+# connects public-inbox processes to PublicInbox::Gcf2::loop()
 package PublicInbox::Gcf2Client;
 use strict;
-use parent 'PublicInbox::Git';
+use parent qw(PublicInbox::DS);
+use PublicInbox::Git;
 use PublicInbox::Spawn qw(popen_rd);
 use IO::Handle ();
+use PublicInbox::Syscall qw(EPOLLONESHOT);
+# fields:
+#      async_cat => GitAsyncCat ref (read-only pipe)
+#      sock => writable pipe to Gcf2::loop
 
-sub new {
+
+sub new  {
        my ($rdr) = @_;
        my $self = bless {}, __PACKAGE__;
+       # ensure the child process has the same @INC we do:
+       my $env = { PERL5LIB => join(':', @INC) };
        my ($out_r, $out_w);
-       pipe($out_r, $out_w) or $self->fail("pipe failed: $!");
+       pipe($out_r, $out_w) or die "pipe failed: $!";
        $rdr //= {};
        $rdr->{0} = $out_r;
-       @$self{qw(in pid)} = popen_rd(['public-inbox-gcf2'], undef, $rdr);
-       $self->{inflight} = [];
-       $self->{out} = $out_w;
+       my $cmd = [$^X, qw[-MPublicInbox::Gcf2 -e PublicInbox::Gcf2::loop()]];
+       @$self{qw(in pid)} = popen_rd($cmd, $env, $rdr);
        fcntl($out_w, 1031, 4096) if $^O eq 'linux'; # 1031: F_SETPIPE_SZ
        $out_w->autoflush(1);
-       $self;
+       $out_w->blocking(0);
+       $self->{inflight} = [];
+       $self->SUPER::new($out_w, EPOLLONESHOT); # detect errors once
 }
 
-sub add_git_dir {
-       my ($self, $git_dir) = @_;
+sub fail {
+       my $self = shift;
+       $self->close; # PublicInbox::DS::close
+       PublicInbox::Git::fail($self, @_);
+}
 
-       # ensure buffers are drained, length($git_dir) may exceed
-       # PIPE_BUF on platforms where PIPE_BUF is only 512 bytes
+sub cat_async ($$$;$) {
+       my ($self, $req, $cb, $arg) = @_;
        my $inflight = $self->{inflight};
-       while (scalar(@$inflight)) {
-               $self->cat_async_step($inflight);
+
+       # {wbuf} is rare, I hope:
+       cat_async_step($self, $inflight) if $self->{wbuf};
+
+       if (!$self->write(\"$req\n")) {
+               $self->fail("gcf2c write: $!") if !$self->{sock};
        }
-       print { $self->{out} } $git_dir, "\n" or
-                               $self->fail("write error: $!");
+       push @$inflight, $req, $cb, $arg;
 }
 
-# always false, since -gcf2 retries internally
+# ensure PublicInbox::Git::cat_async_step never calls cat_async_retry
 sub alternates_changed {}
 
+# this is the write-only end of a pipe, DS->EventLoop will call this
+sub event_step {
+       my ($self) = @_;
+       $self->flush_write;
+       $self->close if !$self->{in}; # process died
+}
+
+no warnings 'once';
+
+# used by GitAsyncCat
+*cat_async_step = \&PublicInbox::Git::cat_async_step;
+
 1;