X-Git-Url: http://www.git.stargrave.org/?a=blobdiff_plain;f=lib%2FPublicInbox%2FLeiToMail.pm;h=f3c039699cc26bdd5c539fe6b3acc8ed5a4c70d7;hb=ec6d2dc31406378f77aa681017083fe8e98b4df9;hp=7e82164631b963157cb53f9bb3d983fad7506c16;hpb=7d2e572aca7297ea2015d2b6e7c71b672521ec82;p=public-inbox.git diff --git a/lib/PublicInbox/LeiToMail.pm b/lib/PublicInbox/LeiToMail.pm index 7e821646..f3c03969 100644 --- a/lib/PublicInbox/LeiToMail.pm +++ b/lib/PublicInbox/LeiToMail.pm @@ -7,27 +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(which spawn popen_rd); -use PublicInbox::LeiDedupe; -use PublicInbox::GitAsyncCat; +use PublicInbox::Spawn qw(spawn); 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 = ( @@ -41,19 +33,21 @@ sub _mbox_hdr_buf ($$$) { my ($eml, $type, $smsg) = @_; $eml->header_set($_) for (qw(Lines Bytes Content-Length)); - # Messages are always 'O' (non-\Recent in IMAP), it saves - # MUAs the trouble of rewriting the mbox if no other - # changes are made - my %hdr = (Status => [ 'O' ]); # set Status, X-Status + my %hdr = (Status => []); # set Status, X-Status for my $k (@{$smsg->{kw} // []}) { 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"; } } - while (my ($name, $chars) = each %hdr) { - $eml->header_set($name, join('', sort @$chars)); + # Messages are always 'O' (non-\Recent in IMAP), it saves + # MUAs the trouble of rewriting the mbox if no other + # changes are made. We put 'O' at the end (e.g. "Status: RO") + # to match mutt(1) output. + $eml->header_set('Status', join('', sort(@{$hdr{Status}})). 'O'); + if (my $chars = delete $hdr{'X-Status'}) { + $eml->header_set('X-Status', join('', sort(@$chars))); } my $buf = delete $eml->{hdr}; @@ -72,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: $!"; @@ -143,18 +137,17 @@ 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"; - } - } my ($write_cb, $smsg) = @$arg; - if ($smsg->{blob} ne $oid) { - die "BUG: expected=$smsg->{blob} got=$oid"; + if ($type eq 'missing' && $smsg->{-lms_ro}) { + if ($bref = $smsg->{-lms_ro}->local_blob($oid, 1)) { + $type = 'blob'; + $size = length($$bref); + } } - $write_cb->($bref, $smsg) if $size > 0; + return warn("W: $oid is $type (!= blob)\n") if $type ne 'blob'; + return warn("E: $oid is empty\n") unless $size; + die "BUG: expected=$smsg->{blob} got=$oid" if $smsg->{blob} ne $oid; + $write_cb->($bref, $smsg); } sub reap_compress { # dwaitpid callback @@ -164,56 +157,10 @@ sub reap_compress { # dwaitpid callback $lei->fail("@$cmd failed", $? >> 8); } -# all of these support -c for stdout and -d for decompression, -# mutt is commonly distributed with hooks for gz, bz2 and xz, at least -# { foo => '' } means "--foo" is passed to the command-line, -# otherwise { foo => '--bar' } passes "--bar" -our %zsfx2cmd = ( - gz => [ qw(GZIP pigz gzip), { rsyncable => '' } ], - bz2 => [ 'bzip2', {} ], - xz => [ 'xz', {} ], - # XXX does anybody care for these? I prefer zstd on entire FSes, - # so it's probably not necessary on a per-file basis - # zst => [ 'zstd', { -default => [ qw(-q) ], # it's noisy by default - # rsyncable => '', threads => '-T' } ], - # zz => [ 'pigz', { -default => [ '--zlib' ], - # rsyncable => '', threads => '-p' }], - # lzo => [ 'lzop', {} ], - # lzma => [ 'lzma', {} ], -); - -sub zsfx2cmd ($$$) { - my ($zsfx, $decompress, $lei) = @_; - my $x = $zsfx2cmd{$zsfx} // die "no support for suffix=.$zsfx"; - my @info = @$x; - my $cmd_opt = pop @info; - my @cmd = (undef, $decompress ? qw(-dc) : qw(-c)); - for my $exe (@info) { - # I think respecting client's ENV{GZIP} is OK, not sure - # about ENV overrides for other, less-common compressors - if ($exe eq uc($exe)) { - $exe = $lei->{env}->{$exe} or next; - } - $cmd[0] = which($exe) and last; - } - $cmd[0] // die join(' or ', @info)." missing for .$zsfx"; - # push @cmd, @{$cmd_opt->{-default}} if $cmd_opt->{-default}; - for my $bool (qw(rsyncable)) { - my $switch = $cmd_opt->{rsyncable} // next; - push @cmd, '--'.($switch || $bool); - } - for my $key (qw(rsyncable)) { # support compression level? - my $switch = $cmd_opt->{$key} // next; - my $val = $lei->{opt}->{$key} // next; - push @cmd, $switch, $val; - } - \@cmd; -} - -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 = zsfx2cmd($zsfx, undef, $lei); + my $cmd = PublicInbox::MboxReader::zsfx2cmd($zsfx, undef, $lei); my ($r, $w) = @{delete $lei->{zpipe}}; my $rdr = { 0 => $r, 1 => $lei->{1}, 2 => $lei->{2} }; my $pid = spawn($cmd, undef, $rdr); @@ -224,18 +171,6 @@ sub _post_augment_mbox { # open a compressor process $lei->{1} = $pp; } -sub decompress_src ($$$) { - my ($in, $zsfx, $lei) = @_; - my $cmd = zsfx2cmd($zsfx, 1, $lei); - popen_rd($cmd, undef, { 0 => $in, 2 => $lei->{2} }); -} - -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) = @_; @@ -260,39 +195,43 @@ sub _mbox_write_cb ($$) { my $atomic_append = !defined($ovv->{lock_path}); my $dedupe = $lei->{dedupe}; $dedupe->prepare_dedupe; + my $lse = $lei->{lse}; # may be undef 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; - my $lk = $ovv->lock_for_scope; - $lei->out($$buf); + if ($atomic_append) { + atomic_append($lei, $buf); + } else { + my $lk = $ovv->lock_for_scope; + $lei->out($$buf); + } + ++$lei->{-nr_write}; } } sub update_kw_maybe ($$$$) { my ($lei, $lse, $eml, $kw) = @_; return unless $lse; - my $x = $lse->kw_changed($eml, $kw); - if ($x) { - $lei->{sto}->ipc_do('set_eml', $eml, { kw => $kw }); - } elsif (!defined($x)) { - $lei->{sto}->ipc_do('set_xkw', $eml, $kw); - } -} - -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, so we extract the OID - # if it looks like one: - 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); - } + my $c = $lse->kw_changed($eml, $kw, my $docids = []); + my $vmd = { kw => $kw }; + 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); + } else { # never-before-seen, import the whole thing + # XXX this is critical in protecting against accidental + # data loss without --augment + $lei->{sto}->ipc_do('set_eml', $eml, $vmd); + } +} + +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); } @@ -304,18 +243,22 @@ sub _rand () { sprintf('%x,%x,%x,%x', rand(0xffffffff), time, $$, ++$seq); } +sub kw2suffix ($;@) { + my $kw = shift; + join('', sort(map { $kw2char{$_} // () } @$kw, @_)); +} + sub _buf2maildir { my ($dst, $buf, $smsg) = @_; my $kw = $smsg->{kw} // []; - my $sfx = join('', sort(map { $kw2char{$_} // () } @$kw)); my $rand = ''; # chosen by die roll :P - my ($tmp, $fh, $final, $ok); + my ($tmp, $fh, $base, $ok); my $common = $smsg->{blob} // _rand; if (defined(my $pct = $smsg->{pct})) { $common .= "=$pct" } 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 @@ -324,11 +267,12 @@ sub _buf2maildir { $dst .= 'cur/'; $rand = ''; do { - $final = $dst.$rand.$common.':2,'.$sfx; - } while (!($ok = link($tmp, $final)) && $! == EEXIST && + $base = $rand.$common.':2,'.kw2suffix($kw); + } while (!($ok = link($tmp, $dst.$base)) && $!{EEXIST} && ($rand = _rand.',')); - die "link($tmp, $final): $!" unless $ok; + die "link($tmp, $dst$base): $!" unless $ok; unlink($tmp) or warn "W: failed to unlink $tmp: $!\n"; + \$base; } else { my $err = "Error writing $smsg->{blob} to $dst: $!\n"; $_[0] = undef; # clobber dst @@ -342,15 +286,19 @@ sub _maildir_write_cb ($$) { my $dedupe = $lei->{dedupe}; $dedupe->prepare_dedupe if $dedupe; my $dst = $lei->{ovv}->{dst}; + my $lse = $lei->{lse}; # may be undef + my $sto = $lei->{opt}->{'mail-sync'} ? $lei->{sto} : undef; + my $out = $sto ? 'maildir:'.$lei->rel2abs($dst) : undef; sub { # for git_to_mail - my ($buf, $smsg, $eml) = @_; + my ($bref, $smsg, $eml) = @_; $dst // return $lei->fail; # dst may be undef-ed in last run - $buf //= \($eml->as_string); - return _buf2maildir($dst, $buf, $smsg) if !$dedupe; - $eml //= PublicInbox::Eml->new($$buf); # copy buf - return if $dedupe->is_dup($eml, $smsg->{blob}); - undef $eml; - _buf2maildir($dst, $buf, $smsg); + return if $dedupe && $dedupe->is_dup($eml // + PublicInbox::Eml->new($$bref), + $smsg); + $lse->xsmsg_vmd($smsg) if $lse; + my $n = _buf2maildir($dst, $bref // \($eml->as_string), $smsg); + $sto->ipc_do('set_sync_info', $smsg->{blob}, $out, $n) if $sto; + ++$lei->{-nr_write}; } } @@ -358,21 +306,50 @@ sub _imap_write_cb ($$) { my ($self, $lei) = @_; my $dedupe = $lei->{dedupe}; $dedupe->prepare_dedupe if $dedupe; - my $imap_append = $lei->{net}->can('imap_append'); - my $mic = $lei->{net}->mic_get($self->{uri}); - my $folder = $self->{uri}->mailbox; + my $append = $lei->{net}->can('imap_append'); + my $uri = $self->{uri}; + my $mic = $lei->{net}->mic_get($uri); + my $folder = $uri->mailbox; + $uri->uidvalidity($mic->uidvalidity($folder)); + my $lse = $lei->{lse}; # may be undef + my $sto = $lei->{opt}->{'mail-sync'} ? $lei->{sto} : undef; sub { # for git_to_mail my ($bref, $smsg, $eml) = @_; - $mic // return $lei->fail; # dst may be undef-ed in last run - if ($dedupe) { - $eml //= PublicInbox::Eml->new($$bref); # copy bref - return if $dedupe->is_dup($eml, $smsg->{blob}); - } - eval { $imap_append->($mic, $folder, $bref, $smsg, $eml) }; + $mic // return $lei->fail; # mic may be undef-ed in last run + return if $dedupe && $dedupe->is_dup($eml // + PublicInbox::Eml->new($$bref), + $smsg); + $lse->xsmsg_vmd($smsg) if $lse; + my $uid = eval { $append->($mic, $folder, $bref, $smsg, $eml) }; if (my $err = $@) { undef $mic; die $err; } + # imap_append returns UID if IMAP server has UIDPLUS extension + ($sto && $uid =~ /\A[0-9]+\z/) and + $sto->ipc_do('set_sync_info', + $smsg->{blob}, $$uri, $uid + 0); + ++$lei->{-nr_write}; + } +} + +sub _text_write_cb ($$) { + my ($self, $lei) = @_; + my $dedupe = $lei->{dedupe}; + $dedupe->prepare_dedupe if $dedupe; + my $lvt = $lei->{lvt}; + my $ovv = $lei->{ovv}; + $lei->{1} // die "no stdout ($ovv->{dst})"; # redirected earlier + $lei->{1}->autoflush(1); + binmode $lei->{1}, ':utf8'; + my $lse = $lei->{lse}; # may be undef + sub { # for git_to_mail + my ($bref, $smsg, $eml) = @_; + $lse->xsmsg_vmd($smsg) if $lse; + $eml //= PublicInbox::Eml->new($bref); # copy bref + return if $dedupe && $dedupe->is_dup($eml, $smsg); + my $lk = $ovv->lock_for_scope; + $lei->out(${$lvt->eml_to_text($smsg, $eml)}, "\n"); } } @@ -396,27 +373,47 @@ sub new { $lei->{ovv}->{dst} = $dst .= '/' if substr($dst, -1) ne '/'; } elsif (substr($fmt, 0, 4) eq 'mbox') { require PublicInbox::MboxReader; - (-d $dst || (-e _ && !-w _)) and die - "$dst exists and is not a writable file\n"; $self->can("eml2$fmt") or die "bad mbox format: $fmt\n"; $self->{base_type} = 'mbox'; - } elsif ($fmt =~ /\Aimaps?\z/) { # TODO .onion support + } elsif ($fmt =~ /\Aimaps?\z/) { 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($lei); 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'; + } elsif ($fmt eq 'text') { + require PublicInbox::LeiViewText; + $lei->{lvt} = PublicInbox::LeiViewText->new($lei); + $self->{base_type} = 'text'; } else { die "bad mail --format=$fmt\n"; } + if ($self->{base_type} =~ /\A(?:text|mbox)\z/) { + (-d $dst || (-e _ && !-w _)) and die + "$dst exists and is not a writable file\n"; + } + if ($self->{base_type} eq 'text') { + my @err = map { + defined($lei->{opt}->{$_}) ? "--$_" : (); + } (qw(mua save)); + die "@err incompatible with $fmt\n" if @err; + } $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; } @@ -430,35 +427,36 @@ sub _pre_augment_maildir { File::Path::mkpath($d); -d $d or die "$d is not a directory"; } + # for utime, so no opendir + open $self->{poke_dh}, '<', "${dst}cur" or die "open ${dst}cur: $!"; } sub _do_augment_maildir { my ($self, $lei) = @_; + return if ($lei->{opt}->{save} // 0) < 0; my $dst = $lei->{ovv}->{dst}; - my $lse = $lei->{sto}->search if $lei->{opt}->{'import-before'}; - my ($mod, $shard) = @{$self->{shard_info} // []}; + my $lse = $lei->{opt}->{'import-before'} ? $lei->{lse} : undef; + 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); } } sub _imap_augment_or_delete { # PublicInbox::NetReader::imap_each cb - my ($url, $uid, $kw, $eml, $lei, $lse, $delete_mic) = @_; + my ($uri, $uid, $kw, $eml, $lei, $lse, $delete_mic) = @_; update_kw_maybe($lei, $lse, $eml, $kw); if ($delete_mic) { - $lei->{net}->imap_delete_1($url, $uid, $delete_mic); + $lei->{net}->imap_delete_1($uri, $uid, $delete_mic); } else { _augment($eml, $lei); } @@ -466,8 +464,9 @@ 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->{sto}->search if $lei->{opt}->{'import-before'}; + my $lse = $lei->{opt}->{'import-before'} ? $lei->{lse} : undef; if ($lei->{opt}->{augment}) { my $dedupe = $lei->{dedupe}; if ($dedupe && $dedupe->prepare_dedupe) { @@ -486,13 +485,38 @@ sub _do_augment_imap { } } +sub _pre_augment_text { + my ($self, $lei) = @_; + my $dst = $lei->{ovv}->{dst}; + my $out; + my $devfd = $lei->path_to_fd($dst) // die "bad $dst"; + if ($devfd >= 0) { + $out = $lei->{$devfd}; + } else { # normal-looking path + if (-p $dst) { + open $out, '>', $dst or die "open($dst): $!"; + } elsif (-f _ || !-e _) { + # text allows augment, HTML/Atom won't + my $mode = $lei->{opt}->{augment} ? '>>' : '>'; + open $out, $mode, $dst or die "open($mode, $dst): $!"; + } else { + die "$dst is not a file or FIFO\n"; + } + } + $lei->{ovv}->ovv_out_lk_init if !$lei->{ovv}->{lock_path}; + $lei->{1} = $out; + undef; +} + sub _pre_augment_mbox { my ($self, $lei) = @_; my $dst = $lei->{ovv}->{dst}; - my $out = $lei->{1}; - if ($dst ne '/dev/stdout') { + my $out; + my $devfd = $lei->path_to_fd($dst) // die "bad $dst"; + if ($devfd >= 0) { + $out = $lei->{$devfd}; + } else { # normal-looking path if (-p $dst) { - $out = undef; open $out, '>', $dst or die "open($dst): $!"; } elsif (-f _ || !-e _) { require PublicInbox::MboxLock; @@ -500,23 +524,24 @@ sub _pre_augment_mbox { PublicInbox::MboxLock->defaults; $self->{mbl} = PublicInbox::MboxLock->acq($dst, 1, $m); $out = $self->{mbl}->{fh}; + } else { + die "$dst is not a file or FIFO\n"; } $lei->{old_1} = $lei->{1}; # keep for spawning MUA } # Perl does SEEK_END even with O_APPEND :< $self->{seekable} = seek($out, 0, SEEK_SET); - if (!$self->{seekable} && $! != ESPIPE && $dst ne '/dev/stdout') { + if (!$self->{seekable} && !$!{ESPIPE} && !defined($devfd)) { die "seek($dst): $!\n"; } if (!$self->{seekable}) { - my $ia = $lei->{opt}->{'import-before'}; + my $imp_before = $lei->{opt}->{'import-before'}; die "--import-before specified but $dst is not seekable\n" - if $ia && !ref($ia); + if $imp_before && !ref($imp_before); die "--augment specified but $dst is not seekable\n" if $lei->{opt}->{augment}; } - state $zsfx_allow = join('|', keys %zsfx2cmd); - if (($self->{zsfx}) = ($dst =~ /\.($zsfx_allow)\z/)) { + if ($self->{zsfx} = PublicInbox::MboxReader::zsfx($dst)) { pipe(my ($r, $w)) or die "pipe: $!"; $lei->{zpipe} = [ $r, $w ]; $lei->{ovv}->{lock_path} and @@ -533,6 +558,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; @@ -540,15 +566,19 @@ sub _do_augment_mbox { truncate($out, 0) or die "truncate($dst): $!"; return; } - my $zsfx = $self->{zsfx}; - my $rd = $zsfx ? decompress_src($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}; $dedupe->prepare_dedupe if $dedupe; } if ($opt->{'import-before'}) { # the default - my $lse = $lei->{sto}->search; + my $lse = $lei->{lse}; PublicInbox::MboxReader->$fmt($rd, \&_mbox_augment_kw_maybe, $lei, $lse, $opt->{augment}); if (!$opt->{augment} and !truncate($out, 0)) { @@ -572,8 +602,8 @@ sub pre_augment { # fast (1 disk seek), runs in same process as post_augment sub do_augment { # slow, runs in wq worker my ($self, $lei) = @_; # _do_augment_maildir, _do_augment_mbox, or _do_augment_imap - my $m = "_do_augment_$self->{base_type}"; - $self->$m($lei); + my $m = $self->can("_do_augment_$self->{base_type}") or return; + $m->($self, $lei); } # fast (spawn compressor or mkdir), runs in same process as pre_augment @@ -586,6 +616,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}; @@ -593,16 +624,16 @@ 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}) { $net->{shard_info} = [ $mod, $shard ]; - } else { # Maildir (MH?) + } else { # Maildir $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) { @@ -612,6 +643,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]; @@ -622,39 +654,49 @@ sub do_post_auth { sub ipc_atfork_child { my ($self) = @_; my $lei = $self->{lei}; - $lei->lei_atfork_child; + $lei->_lei_atfork_child; + if (my $lse = $lei->{lse}) { + $self->{-lms_ro} = $lse->{-lms_ro} //= $lse->lms; + } $lei->{auth}->do_auth_atfork($self) if $lei->{auth}; $SIG{__WARN__} = PublicInbox::Eml::warn_ignore_cb(); $self->SUPER::ipc_atfork_child; } sub lock_free { - $_[0]->{base_type} =~ /\A(?:maildir|mh|imap|jmap)\z/ ? 1 : 0; + $_[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') { my $t = time + 1; - utime($t, $t, $self->{dst} . 'cur'); + utime($t, $t, $self->{poke_dh}) or warn "futimes: $!"; } } 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; + $smsg->{-lms_ro} = $self->{-lms_ro}; + $self->{lei}->{ale}->git->cat_async($smsg->{blob}, \&git_to_mail, [$self->{wcb}, $smsg]); } sub wq_atexit_child { my ($self) = @_; delete $self->{wcb}; - $self->{lei}->{ale}->git->async_wait_all; - $SIG{__WARN__} = 'DEFAULT'; + my $lei = $self->{lei}; + $lei->{ale}->git->async_wait_all; + my $nr = delete($lei->{-nr_write}) or return; + return if $lei->{early_mua} || !$lei->{-progress} || !$lei->{pkt_op_p}; + require PublicInbox::PktOp; + PublicInbox::PktOp::pkt_do($lei->{pkt_op_p}, 'l2m_progress', $nr); } # called in top-level lei-daemon when LeiAuth is done -sub net_merge_complete { +sub net_merge_all_done { my ($self) = @_; $self->wq_broadcast('do_post_auth'); $self->wq_close(1);