From: Eric Wong Date: Sat, 3 Apr 2021 02:24:24 +0000 (+0000) Subject: lei tag: fix tagging of IMAP inputs X-Git-Tag: v1.7.0~836 X-Git-Url: http://www.git.stargrave.org/?a=commitdiff_plain;h=f7cf8f06088b417dc3fe5bdda228132a7ed2c4ff;p=public-inbox.git lei tag: fix tagging of IMAP inputs We need net_merge_all and to lock the number of worker jobs. Parallel inputs are not supported, yet (is it needed?, I don't expect this to be used for multiple files very often...). --- diff --git a/lib/PublicInbox/LeiTag.pm b/lib/PublicInbox/LeiTag.pm index d572a84a..06313139 100644 --- a/lib/PublicInbox/LeiTag.pm +++ b/lib/PublicInbox/LeiTag.pm @@ -112,7 +112,8 @@ sub lei_tag { # the "lei tag" method my $ops = { '' => [ \&tag_done, $lei ] }; $lei->{auth}->op_merge($ops, $self) if $lei->{auth}; $self->{vmd_mod} = $vmd_mod; - (my $op_c, $ops) = $lei->workers_start($self, 'lei_tag', 1, $ops); + my $j = $self->{-wq_nr_workers} = 1; # locked for now + (my $op_c, $ops) = $lei->workers_start($self, 'lei_tag', $j, $ops); $lei->{tag} = $self; net_merge_complete($self) unless $lei->{auth}; $op_c->op_wait_event($ops); @@ -175,4 +176,7 @@ sub _complete_tag { } grep(/$re\Q$cur/, @all); } +no warnings 'once'; # the following works even when LeiAuth is lazy-loaded +*net_merge_all = \&PublicInbox::LeiAuth::net_merge_all; + 1; diff --git a/t/lei-import-imap.t b/t/lei-import-imap.t index fd38037a..7e4d44b9 100644 --- a/t/lei-import-imap.t +++ b/t/lei-import-imap.t @@ -23,5 +23,6 @@ test_lei({ tmpdir => $tmpdir }, sub { my %r; for (@$out) { $r{ref($_)}++ } is_deeply(\%r, { 'HASH' => scalar(@$out) }, 'all hashes'); + lei_ok([qw(tag +kw:seen), "imap://$host_port/t.v2.0"], undef, undef); }); done_testing;