X-Git-Url: http://www.git.stargrave.org/?a=blobdiff_plain;f=lib%2FPublicInbox%2FLeiToMail.pm;h=0fa0bd9a6a2ef124724b085be5677b5395738ecb;hb=91ae332610370d2a5da2cfb0bd1dff09463e5438;hp=4c33c7529a1042f0a58eb0f2540756fb2eb144da;hpb=5565abd5ca88deec5cf505d2fb75ad5bac54bbcf;p=public-inbox.git diff --git a/lib/PublicInbox/LeiToMail.pm b/lib/PublicInbox/LeiToMail.pm index 4c33c752..0fa0bd9a 100644 --- a/lib/PublicInbox/LeiToMail.pm +++ b/lib/PublicInbox/LeiToMail.pm @@ -7,28 +7,19 @@ use strict; use v5.10.1; use parent qw(PublicInbox::IPC); use PublicInbox::Eml; -use PublicInbox::Lock; use PublicInbox::ProcessPipe; use PublicInbox::Spawn qw(spawn); -use PublicInbox::LeiDedupe; -use PublicInbox::Git; -use PublicInbox::GitAsyncCat; use PublicInbox::PktOp qw(pkt_do); use Symbol qw(gensym); use IO::Handle; # ->autoflush use Fcntl qw(SEEK_SET SEEK_END O_CREAT O_EXCL O_WRONLY); -use Errno qw(EEXIST ESPIPE ENOENT EPIPE); -use Digest::SHA qw(sha256_hex); - -# struggles with short-lived repos, Gcf2Client makes little sense with lei; -# but we may use in-process libgit2 in the future. -$PublicInbox::GitAsyncCat::GCF2C = 0; my %kw2char = ( # Maildir characters draft => 'D', flagged => 'F', + forwarded => 'P', # passed answered => 'R', - seen => 'S' + seen => 'S', ); my %kw2status = ( @@ -47,7 +38,7 @@ sub _mbox_hdr_buf ($$$) { if (my $ent = $kw2status{$k}) { push @{$hdr{$ent->[0]}}, $ent->[1]; } else { # X-Label? - warn "TODO: keyword `$k' not supported for mbox\n"; + warn "# keyword `$k' not supported for mbox\n"; } } # Messages are always 'O' (non-\Recent in IMAP), it saves @@ -75,7 +66,7 @@ sub atomic_append { # for on-disk destinations (O_APPEND, or O_EXCL) if (defined(my $w = syswrite($lei->{1} // return, $$buf))) { return if $w == length($$buf); $buf = "short atomic write: $w != ".length($$buf); - } elsif ($! == EPIPE) { + } elsif ($!{EPIPE}) { return $lei->note_sigpipe(1); } else { $buf = "atomic write: $!"; @@ -146,18 +137,11 @@ sub eml2mboxcl2 { sub git_to_mail { # git->cat_async callback my ($bref, $oid, $type, $size, $arg) = @_; - if ($type ne 'blob') { - if ($type eq 'missing') { - warn "missing $oid\n"; - } else { - warn "unexpected type=$type for $oid\n"; - } - } + return warn("W: $oid is $type (!= blob)\n") if $type ne 'blob'; + return warn("E: $oid is empty\n") unless $size; my ($write_cb, $smsg) = @$arg; - if ($smsg->{blob} ne $oid) { - die "BUG: expected=$smsg->{blob} got=$oid"; - } - $write_cb->($bref, $smsg) if $size > 0; + die "BUG: expected=$smsg->{blob} got=$oid" if $smsg->{blob} ne $oid; + $write_cb->($bref, $smsg); } sub reap_compress { # dwaitpid callback @@ -167,7 +151,7 @@ sub reap_compress { # dwaitpid callback $lei->fail("@$cmd failed", $? >> 8); } -sub _post_augment_mbox { # open a compressor process +sub _post_augment_mbox { # open a compressor process from top-level process my ($self, $lei) = @_; my $zsfx = $self->{zsfx} or return; my $cmd = PublicInbox::MboxReader::zsfx2cmd($zsfx, undef, $lei); @@ -181,12 +165,6 @@ sub _post_augment_mbox { # open a compressor process $lei->{1} = $pp; } -sub dup_src ($) { - my ($in) = @_; - open my $dup, '+>>&', $in or die "dup: $!"; - $dup; -} - # --augment existing output destination, with deduplication sub _augment { # MboxReader eml_cb my ($eml, $lei) = @_; @@ -215,7 +193,7 @@ sub _mbox_write_cb ($$) { sub { # for git_to_mail my ($buf, $smsg, $eml) = @_; $eml //= PublicInbox::Eml->new($buf); - return if $dedupe->is_dup($eml, $smsg->{blob}); + return if $dedupe->is_dup($eml, $smsg); $lse->xsmsg_vmd($smsg) if $lse; $buf = $eml2mbox->($eml, $smsg); return atomic_append($lei, $buf) if $atomic_append; @@ -227,10 +205,10 @@ sub _mbox_write_cb ($$) { sub update_kw_maybe ($$$$) { my ($lei, $lse, $eml, $kw) = @_; return unless $lse; - my $lse_oids = $lse->kw_changed($eml, $kw); + my $c = $lse->kw_changed($eml, $kw, my $docids = []); my $vmd = { kw => $kw }; - if ($lse_oids) { # already in lei/store - $lei->{sto}->ipc_do('set_eml', $eml, $vmd); + if (scalar @$docids) { # already in lei/store + $lei->{sto}->ipc_do('set_eml_vmd', undef, $vmd, $docids) if $c; } elsif (my $xoids = $lei->{ale}->xoids_for($eml)) { # it's in an external, only set kw, here $lei->{sto}->ipc_do('set_xvmd', $xoids, $eml, $vmd); @@ -241,17 +219,9 @@ sub update_kw_maybe ($$$$) { } } -sub _augment_or_unlink { # maildir_each_eml cb - my ($f, $kw, $eml, $lei, $lse, $mod, $shard, $unlink) = @_; - if ($mod) { - # can't get dirent.d_ino w/ pure Perl readdir, so we extract - # the OID if it looks like one instead of doing stat(2) - my $hex = $f =~ m!\b([a-f0-9]{40,})[^/]*\z! ? - $1 : sha256_hex($f); - my $recno = hex(substr($hex, 0, 8)); - return if ($recno % $mod) != $shard; - update_kw_maybe($lei, $lse, $eml, $kw); - } +sub _md_update { # maildir_each_eml cb + my ($f, $kw, $eml, $lei, $lse, $unlink) = @_; + update_kw_maybe($lei, $lse, $eml, $kw); $unlink ? unlink($f) : _augment($eml, $lei); } @@ -274,7 +244,7 @@ sub _buf2maildir { do { $tmp = $dst.'tmp/'.$rand.$common; } while (!($ok = sysopen($fh, $tmp, O_CREAT|O_EXCL|O_WRONLY)) && - $! == EEXIST && ($rand = _rand.',')); + $!{EEXIST} && ($rand = _rand.',')); if ($ok && print $fh $$buf and close($fh)) { # ignore new/ and write only to cur/, otherwise MUAs # with R/W access to the Maildir will end up doing @@ -284,7 +254,7 @@ sub _buf2maildir { $rand = ''; do { $final = $dst.$rand.$common.':2,'.$sfx; - } while (!($ok = link($tmp, $final)) && $! == EEXIST && + } while (!($ok = link($tmp, $final)) && $!{EEXIST} && ($rand = _rand.',')); die "link($tmp, $final): $!" unless $ok; unlink($tmp) or warn "W: failed to unlink $tmp: $!\n"; @@ -309,7 +279,7 @@ sub _maildir_write_cb ($$) { $lse->xsmsg_vmd($smsg) if $lse; return _buf2maildir($dst, $buf, $smsg) if !$dedupe; $eml //= PublicInbox::Eml->new($$buf); # copy buf - return if $dedupe->is_dup($eml, $smsg->{blob}); + return if $dedupe->is_dup($eml, $smsg); undef $eml; _buf2maildir($dst, $buf, $smsg); } @@ -325,10 +295,10 @@ sub _imap_write_cb ($$) { my $lse = $lei->{lse}; # may be undef sub { # for git_to_mail my ($bref, $smsg, $eml) = @_; - $mic // return $lei->fail; # dst may be undef-ed in last run + $mic // return $lei->fail; # mic may be undef-ed in last run if ($dedupe) { $eml //= PublicInbox::Eml->new($$bref); # copy bref - return if $dedupe->is_dup($eml, $smsg->{blob}); + return if $dedupe->is_dup($eml, $smsg); } $lse->xsmsg_vmd($smsg) if $lse; eval { $imap_append->($mic, $folder, $bref, $smsg, $eml) }; @@ -365,21 +335,29 @@ sub new { $self->{base_type} = 'mbox'; } elsif ($fmt =~ /\Aimaps?\z/) { # TODO .onion support require PublicInbox::NetWriter; + require PublicInbox::URIimap; my $net = PublicInbox::NetWriter->new; - $net->add_url($dst); $net->{quiet} = $lei->{opt}->{quiet}; - my $err = $net->errors($dst); + my $uri = PublicInbox::URIimap->new($dst)->canonical; + $net->add_url($uri); + my $err = $net->errors; return $lei->fail($err) if $err; - require PublicInbox::URIimap; # TODO: URI cast early - $self->{uri} = PublicInbox::URIimap->new($dst); - $self->{uri}->mailbox or die "No mailbox: $dst"; + $uri->mailbox or return $lei->fail("No mailbox: $dst"); + $self->{uri} = $uri; + $dst = $lei->{ovv}->{dst} = $$uri; # canonicalized $lei->{net} = $net; $self->{base_type} = 'imap'; } else { die "bad mail --format=$fmt\n"; } $self->{dst} = $dst; - $lei->{dedupe} = PublicInbox::LeiDedupe->new($lei); + $lei->{dedupe} = $lei->{lss} // do { + my $dd_cls = 'PublicInbox::'. + ($lei->{opt}->{save} ? 'LeiSavedSearch' : 'LeiDedupe'); + eval "require $dd_cls"; + die "$dd_cls: $@" if $@; + $dd_cls->new($lei); + }; $self; } @@ -397,23 +375,22 @@ sub _pre_augment_maildir { sub _do_augment_maildir { my ($self, $lei) = @_; + return if ($lei->{opt}->{save} // 0) < 0; my $dst = $lei->{ovv}->{dst}; my $lse = $lei->{opt}->{'import-before'} ? $lei->{lse} : undef; - my ($mod, $shard) = @{$self->{shard_info} // []}; + my $mdr = PublicInbox::MdirReader->new; if ($lei->{opt}->{augment}) { my $dedupe = $lei->{dedupe}; if ($dedupe && $dedupe->prepare_dedupe) { - PublicInbox::MdirReader::maildir_each_eml($dst, - \&_augment_or_unlink, - $lei, $lse, $mod, $shard); + $mdr->{shard_info} = $self->{shard_info}; + $mdr->maildir_each_eml($dst, \&_md_update, $lei, $lse); $dedupe->pause_dedupe; } } elsif ($lse) { - PublicInbox::MdirReader::maildir_each_eml($dst, - \&_augment_or_unlink, - $lei, $lse, $mod, $shard, 1); + $mdr->{shard_info} = $self->{shard_info}; + $mdr->maildir_each_eml($dst, \&_md_update, $lei, $lse, 1); } else {# clobber existing Maildir - PublicInbox::MdirReader::maildir_each_file($dst, \&_unlink); + $mdr->maildir_each_file($dst, \&_unlink); } } @@ -429,6 +406,7 @@ sub _imap_augment_or_delete { # PublicInbox::NetReader::imap_each cb sub _do_augment_imap { my ($self, $lei) = @_; + return if ($lei->{opt}->{save} // 0) < 0; my $net = $lei->{net}; my $lse = $lei->{opt}->{'import-before'} ? $lei->{lse} : undef; if ($lei->{opt}->{augment}) { @@ -472,7 +450,7 @@ sub _pre_augment_mbox { } # Perl does SEEK_END even with O_APPEND :< $self->{seekable} = seek($out, 0, SEEK_SET); - if (!$self->{seekable} && $! != ESPIPE && !defined($devfd)) { + if (!$self->{seekable} && !$!{ESPIPE} && !defined($devfd)) { die "seek($dst): $!\n"; } if (!$self->{seekable}) { @@ -499,6 +477,7 @@ sub _do_augment_mbox { my ($self, $lei) = @_; return unless $self->{seekable}; my $opt = $lei->{opt}; + return if ($opt->{save} // 0) < 0; my $out = $lei->{1}; my ($fmt, $dst) = @{$lei->{ovv}}{qw(fmt dst)}; return unless -s $out; @@ -506,9 +485,12 @@ sub _do_augment_mbox { truncate($out, 0) or die "truncate($dst): $!"; return; } - my $zsfx = $self->{zsfx}; - my $rd = $zsfx ? PublicInbox::MboxReader::zsfxcat($out, $zsfx, $lei) - : dup_src($out); + my $rd; + if (my $zsfx = $self->{zsfx}) { + $rd = PublicInbox::MboxReader::zsfxcat($out, $zsfx, $lei); + } else { + open($rd, '+>>&', $out) or die "dup: $!"; + } my $dedupe; if ($opt->{augment}) { $dedupe = $lei->{dedupe}; @@ -553,6 +535,7 @@ sub post_augment { $m->($self, $lei, @args); } +# called by every single l2m worker process sub do_post_auth { my ($self) = @_; my $lei = $self->{lei}; @@ -560,7 +543,7 @@ sub do_post_auth { pkt_do($lei->{pkt_op_p}, 'incr_start_query') or die "incr_start_query: $!"; my $aug; - if (lock_free($self)) { + if (lock_free($self)) { # all workers do_augment my $mod = $self->{-wq_nr_workers}; my $shard = $self->{-wq_worker_nr}; if (my $net = $lei->{net}) { @@ -569,7 +552,7 @@ sub do_post_auth { $self->{shard_info} = [ $mod, $shard ]; } $aug = '+'; # incr_post_augment - } elsif ($self->{-wq_worker_nr} == 0) { + } elsif ($self->{-wq_worker_nr} == 0) { # 1st worker do_augment $aug = '.'; # do_post_augment } if ($aug) { @@ -579,6 +562,7 @@ sub do_post_auth { pkt_do($lei->{pkt_op_p}, $aug) == 1 or die "do_post_augment trigger: $!"; } + # done augmenting, connect the compressor pipe for each worker if (my $zpipe = delete $lei->{zpipe}) { $lei->{1} = $zpipe->[1]; close $zpipe->[0]; @@ -599,6 +583,7 @@ sub lock_free { $_[0]->{base_type} =~ /\A(?:maildir|imap|jmap)\z/ ? 1 : 0; } +# wakes up the MUA when complete so it can refresh messages list sub poke_dst { my ($self) = @_; if ($self->{base_type} eq 'maildir') { @@ -608,8 +593,9 @@ sub poke_dst { } sub write_mail { # via ->wq_io_do - my ($self, $smsg) = @_; - git_async_cat($self->{lei}->{ale}->git, $smsg->{blob}, \&git_to_mail, + my ($self, $smsg, $eml) = @_; + return $self->{wcb}->(undef, $smsg, $eml) if $eml; + $self->{lei}->{ale}->git->cat_async($smsg->{blob}, \&git_to_mail, [$self->{wcb}, $smsg]); } @@ -617,7 +603,6 @@ sub wq_atexit_child { my ($self) = @_; delete $self->{wcb}; $self->{lei}->{ale}->git->async_wait_all; - $SIG{__WARN__} = 'DEFAULT'; } # called in top-level lei-daemon when LeiAuth is done