]> Sergey Matveev's repositories - public-inbox.git/commitdiff
lei: replace --thread with --threads
authorEric Wong <e@80x24.org>
Sun, 7 Feb 2021 08:51:56 +0000 (08:51 +0000)
committerEric Wong <e@80x24.org>
Sun, 7 Feb 2021 22:57:15 +0000 (22:57 +0000)
Nobody is expected to use long options, but for consistency
with mairix(1), we'll use the pluralized option throughout
(including existing PublicInbox::{Search,SearchView}).

Link: https://public-inbox.org/meta/20210206090119.GA14519@dcvr/
Documentation/lei-q.pod
lib/PublicInbox/LEI.pm
lib/PublicInbox/LeiHelp.pm
lib/PublicInbox/LeiQuery.pm
lib/PublicInbox/LeiXSearch.pm
lib/PublicInbox/Mbox.pm
lib/PublicInbox/Search.pm
lib/PublicInbox/SearchView.pm

index 07c742d20d4512f1d89e084eb0def23731b72f1d..8f053a55bf60e71ab3ec7a02ca333faa2c734b62 100644 (file)
@@ -47,7 +47,7 @@ or C<neomutt>.
 
 Augment output destination instead of clobbering it.
 
-=item -t, --thread
+=item -t, --threads
 
 Return all messages in the same thread as the actual match(es).
 
index 818f2cfbcd16ae3d9920041fb247df06f417cf13..31e6b4a8e407a01a7fb56379b480fe3078c216d8 100644 (file)
@@ -109,14 +109,14 @@ sub index_opt {
 # command => [ positional_args, 1-line description, Getopt::Long option spec ]
 our %CMD = ( # sorted in order of importance/use:
 'q' => [ '--stdin|SEARCH_TERMS...', 'search for messages matching terms', qw(
-       save-as=s output|mfolder|o=s format|f=s dedupe|d=s thread|t augment|a
+       save-as=s output|mfolder|o=s format|f=s dedupe|d=s threads|t augment|a
        sort|s=s reverse|r offset=i remote! local! external! pretty
        include|I=s@ exclude=s@ only=s@ jobs|j=s globoff|g stdin|
        mua=s no-torsocks torsocks=s verbose|v+ quiet|q),
        PublicInbox::LeiQuery::curl_opt(), opt_dash('limit|n=i', '[0-9]+') ],
 
 'show' => [ 'MID|OID', 'show a given object (Message-ID or object ID)',
-       qw(type=s solve! format|f=s dedupe|d=s thread|t remote local!),
+       qw(type=s solve! format|f=s dedupe|d=s threads|t remote local!),
        pass_through('git show') ],
 
 'add-external' => [ 'LOCATION',
@@ -135,9 +135,9 @@ our %CMD = ( # sorted in order of importance/use:
 'rm-query' => [ 'QUERY_NAME', 'remove a saved search' ],
 'mv-query' => [ qw(OLD_NAME NEW_NAME), 'rename a saved search' ],
 
-'plonk' => [ '--thread|--from=IDENT',
-       'exclude mail matching From: or thread from non-Message-ID searches',
-       qw(stdin| thread|t from|f=s mid=s oid=s) ],
+'plonk' => [ '--threads|--from=IDENT',
+       'exclude mail matching From: or threads from non-Message-ID searches',
+       qw(stdin| threads|t from|f=s mid=s oid=s) ],
 'mark' => [ 'MESSAGE_FLAGS...',
        'set/unset keywords on message(s) from stdin',
        qw(stdin| oid=s exact by-mid|mid:s) ],
@@ -224,9 +224,9 @@ my %OPTDESC = (
 
 'dedupe|d=s' => ['STRATEGY|content|oid|mid|none',
                'deduplication strategy'],
-'show  thread|t' => 'display entire thread a message belongs to',
-'q     thread|t' =>
-       'return all messages in the same thread as the actual match(es)',
+'show  threads|t' => 'display entire thread a message belongs to',
+'q     threads|t' =>
+       'return all messages in the same threads as the actual match(es)',
 'augment|a' => 'augment --output destination instead of clobbering',
 
 'output|mfolder|o=s' => [ 'MFOLDER',
index e62298f7d8176b9595235c479174ce6db1d5a6d5..a654e1c2bb2281b737ac9e422881765d2d6d7627 100644 (file)
@@ -40,7 +40,7 @@ sub call {
                        @vals = (' [', undef, ']');
                } elsif ($x =~ s/=.+//) { # required arg: $x = "type=s"
                        @vals = (' ', undef);
-               } # else: no args $x = 'thread|t'
+               } # else: no args $x = 'threads|t'
 
                # we support underscore options from public-inbox-* commands;
                # but they've never been documented and will likely go away.
@@ -48,7 +48,7 @@ sub call {
                for (grep { !/_/ && !$NOHELP{$_} } split(/\|/, $x)) {
                        length($_) > 1 ? push(@l, "--$_") : push(@s, "-$_");
                }
-               if (!scalar(@vals)) { # no args 'thread|t'
+               if (!scalar(@vals)) { # no args 'threads|t'
                } elsif ($arg_vals =~ s/\A([A-Z_]+)\b//) { # "NAME"
                        $vals[1] = $1;
                } else {
index 0346498fa4f50851f225a84fbe357233d2d1b3c5..9a6fa7181c6abd83fde876ca55784313151077ec 100644 (file)
@@ -81,7 +81,7 @@ sub lei_q {
        $self->{l2m}->{jobs} = ($mj // $nproc) if $self->{l2m};
        PublicInbox::LeiOverview->new($self) or return;
 
-       my %mset_opt = map { $_ => $opt->{$_} } qw(thread limit offset);
+       my %mset_opt = map { $_ => $opt->{$_} } qw(threads limit offset);
        $mset_opt{asc} = $opt->{'reverse'} ? 1 : 0;
        $mset_opt{limit} //= 10000;
        if (defined(my $sort = $opt->{'sort'})) {
@@ -96,7 +96,7 @@ sub lei_q {
                }
        }
        # descending docid order
-       $mset_opt{relevance} //= -2 if $opt->{thread};
+       $mset_opt{relevance} //= -2 if $opt->{threads};
        $self->{mset_opt} = \%mset_opt;
 
        if ($opt->{stdin}) {
index 1024b0209da05a124b7cdd4a01b34d04fcc8b272..2794140a58998b6390f9f5b0a5a437716db7e282 100644 (file)
@@ -118,7 +118,7 @@ sub mset_progress {
        }
 }
 
-sub query_thread_mset { # for --thread
+sub query_thread_mset { # for --threads
        my ($self, $ibxish) = @_;
        local $0 = "$0 query_thread_mset";
        my $lei = $self->{lei};
@@ -151,7 +151,7 @@ sub query_thread_mset { # for --thread
        $lei->{ovv}->ovv_atexit_child($lei);
 }
 
-sub query_mset { # non-parallel for non-"--thread" users
+sub query_mset { # non-parallel for non-"--threads" users
        my ($self) = @_;
        local $0 = "$0 query_mset";
        my $lei = $self->{lei};
@@ -204,7 +204,7 @@ sub query_remote_mboxrd {
        my $lei = $self->{lei};
        my ($opt, $env) = @$lei{qw(opt env)};
        my @qform = (q => $lei->{mset_opt}->{qstr}, x => 'm');
-       push(@qform, t => 1) if $opt->{thread};
+       push(@qform, t => 1) if $opt->{threads};
        my $verbose = $opt->{verbose};
        my ($reap_tail, $reap_curl);
        my $cerr = File::Temp->new(TEMPLATE => 'curl.err-XXXX', TMPDIR => 1);
@@ -326,7 +326,7 @@ my $MAX_PER_HOST = 4;
 
 sub concurrency {
        my ($self, $opt) = @_;
-       my $nl = $opt->{thread} ? locals($self) : 1;
+       my $nl = $opt->{threads} ? locals($self) : 1;
        my $nr = remotes($self);
        $nr = $MAX_PER_HOST if $nr > $MAX_PER_HOST;
        $nl + $nr;
@@ -337,7 +337,7 @@ sub start_query { # always runs in main (lei-daemon) process
        if (my $l2m = $lei->{l2m}) {
                $lei->start_mua if $l2m->lock_free;
        }
-       if ($lei->{opt}->{thread}) {
+       if ($lei->{opt}->{threads}) {
                for my $ibxish (locals($self)) {
                        $self->wq_io_do('query_thread_mset', [], $ibxish);
                }
@@ -393,7 +393,7 @@ sub do_query {
                # 1031: F_SETPIPE_SZ
                fcntl($lei->{startq}, 1031, 4096) if $^O eq 'linux';
        }
-       if (!$lei->{opt}->{thread} && locals($self)) { # for query_mset
+       if (!$lei->{opt}->{threads} && locals($self)) { # for query_mset
                # lei->{git_tmp} is set for wq_wait_old so we don't
                # delete until all lei2mail + lei_xsearch workers are reaped
                $lei->{git_tmp} = $self->{git_tmp} = git_tmp($self);
index 964147fa9667ebf4ead380ff55719fd20c33a3fe..1fca356b60ea6a6429bf246ffeefe2fd7f3f5421 100644 (file)
@@ -236,7 +236,7 @@ sub mbox_all {
                return PublicInbox::WWW::need($ctx, 'Overview');
 
        my $qopts = $ctx->{qopts} = { relevance => -1 }; # ORDER BY docid ASC
-       $qopts->{thread} = 1 if $q->{t};
+       $qopts->{threads} = 1 if $q->{t};
        my $mset = $srch->mset($q_string, $qopts);
        $qopts->{offset} = $mset->size or
                        return [404, [qw(Content-Type text/plain)],
index 7c6a16bec9a1fa87e57c304cf2b1c7a4d635cec0..dbae3bc57d2031df3fae6280418d641995ddff37 100644 (file)
@@ -336,7 +336,7 @@ sub _enquire_once { # retry_reopen callback
        }
 
        # `mairix -t / --threads' or JMAP collapseThreads
-       if ($opts->{thread} && has_threadid($self)) {
+       if ($opts->{threads} && has_threadid($self)) {
                $enquire->set_collapse_key(THREADID);
        }
        $enquire->get_mset($opts->{offset} || 0, $opts->{limit} || 50);
index d50d3cf6c6188a4199b8437b71f9aae74a5e623e..08c77f35000e00d878746576599123bb377ffaed 100644 (file)
@@ -48,7 +48,7 @@ sub sres_top_html {
                limit => $q->{l},
                offset => $o,
                relevance => $q->{r},
-               thread => $q->{t},
+               threads => $q->{t},
                asc => $asc,
        };
        my ($mset, $total, $err, $html);