[PATCH 0/2] doc: lei manpages for remaining commands

[PATCH 0/2] doc: lei manpages for remaining commands

[Kyle Meyer]

This lei manpage update should take care of the remaining lei commands
(aside from -help and -sucks) that don't have manpages.

  [1/2] doc: lei: restore alphabetical order to some listings
  [2/2] doc: lei: add manpages for remaining commands

 Documentation/lei-add-watch.pod         | 33 ++++++++++++++
 Documentation/lei-forget-mail-sync.pod  | 31 +++++++++++++
 Documentation/lei-inspect.pod           | 57 ++++++++++++++++++++++++
 Documentation/lei-ls-mail-source.pod    | 58 +++++++++++++++++++++++++
 Documentation/lei-ls-mail-sync.pod      |  2 +-
 Documentation/lei-ls-watch.pod          | 29 +++++++++++++
 Documentation/lei-mail-diff.pod         | 33 ++++++++++++++
 Documentation/lei-refresh-mail-sync.pod |  2 +-
 Documentation/lei-rm-watch.pod          | 30 +++++++++++++
 Documentation/lei.pod                   | 14 ++++++
 Documentation/txt2pre                   |  9 +++-
 MANIFEST                                |  7 +++
 Makefile.PL                             | 14 +++---
 13 files changed, 309 insertions(+), 10 deletions(-)
 create mode 100644 Documentation/lei-add-watch.pod
 create mode 100644 Documentation/lei-forget-mail-sync.pod
 create mode 100644 Documentation/lei-inspect.pod
 create mode 100644 Documentation/lei-ls-mail-source.pod
 create mode 100644 Documentation/lei-ls-watch.pod
 create mode 100644 Documentation/lei-mail-diff.pod
 create mode 100644 Documentation/lei-rm-watch.pod


base-commit: 299b40d252cf4d4db6fa29ad18cb78777f1f55fc
-- 
2.33.0

[PATCH 1/2] doc: lei: restore alphabetical order to some listings

[Kyle Meyer]

Most the lei-related entries in txt2pre and Makefile.PL are in
alphabetical order.  Reorder the few that aren't.

While at it, reflow the Makefile.PL entries in preparation for the
entries that will be added in the next commit.
---
 Documentation/txt2pre |  2 +-
 Makefile.PL           | 14 +++++++-------
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/Documentation/txt2pre b/Documentation/txt2pre
index 7bc31d23..bc94d404 100755
--- a/Documentation/txt2pre
+++ b/Documentation/txt2pre
@@ -33,8 +33,8 @@ for (qw[lei(1)
 	lei-q(1)
 	lei-rediff(1)
 	lei-rm(1)
-	lei-store-format(5)
 	lei-security(7)
+	lei-store-format(5)
 	lei-tag(1)
 	lei-up(1)
 	public-inbox.cgi(1)
diff --git a/Makefile.PL b/Makefile.PL
index c41c408a..22a58bee 100644
--- a/Makefile.PL
+++ b/Makefile.PL
@@ -45,13 +45,13 @@ $v->{-m1} = [ map {
 		}
 	} @EXE_FILES,
 	qw(
-	lei-add-external lei-blob lei-config lei-convert lei-edit-search
-	lei-export-kw
-	lei-daemon-kill lei-daemon-pid lei-forget-external lei-forget-search
-	lei-import lei-index lei-init lei-lcat lei-ls-external lei-ls-label
-	lei-ls-mail-sync lei-ls-search lei-p2q lei-q lei-rediff
-	lei-refresh-mail-sync
-	lei-rm lei-tag
+	lei-add-external lei-blob lei-config lei-convert
+	lei-daemon-kill lei-daemon-pid lei-edit-search lei-export-kw
+	lei-forget-external lei-forget-search
+	lei-import lei-index lei-init lei-lcat
+	lei-ls-external lei-ls-label lei-ls-mail-sync
+	lei-ls-search lei-p2q lei-q
+	lei-rediff lei-refresh-mail-sync lei-rm lei-tag
 	lei-up)];
 $v->{-m5} = [ qw(public-inbox-config public-inbox-v1-format
 		public-inbox-v2-format public-inbox-extindex-format
-- 
2.33.0

[PATCH 2/2] doc: lei: add manpages for remaining commands

[Kyle Meyer]

At this point all of the current lei commands, aside from -help and
-sucks, should be covered.
---
 Documentation/lei-add-watch.pod         | 33 ++++++++++++++
 Documentation/lei-forget-mail-sync.pod  | 31 +++++++++++++
 Documentation/lei-inspect.pod           | 57 ++++++++++++++++++++++++
 Documentation/lei-ls-mail-source.pod    | 58 +++++++++++++++++++++++++
 Documentation/lei-ls-mail-sync.pod      |  2 +-
 Documentation/lei-ls-watch.pod          | 29 +++++++++++++
 Documentation/lei-mail-diff.pod         | 33 ++++++++++++++
 Documentation/lei-refresh-mail-sync.pod |  2 +-
 Documentation/lei-rm-watch.pod          | 30 +++++++++++++
 Documentation/lei.pod                   | 14 ++++++
 Documentation/txt2pre                   |  7 +++
 MANIFEST                                |  7 +++
 Makefile.PL                             | 12 ++---
 13 files changed, 307 insertions(+), 8 deletions(-)
 create mode 100644 Documentation/lei-add-watch.pod
 create mode 100644 Documentation/lei-forget-mail-sync.pod
 create mode 100644 Documentation/lei-inspect.pod
 create mode 100644 Documentation/lei-ls-mail-source.pod
 create mode 100644 Documentation/lei-ls-watch.pod
 create mode 100644 Documentation/lei-mail-diff.pod
 create mode 100644 Documentation/lei-rm-watch.pod

diff --git a/Documentation/lei-add-watch.pod b/Documentation/lei-add-watch.pod
new file mode 100644
index 00000000..60984618
--- /dev/null
+++ b/Documentation/lei-add-watch.pod
@@ -0,0 +1,33 @@
+=head1 NAME
+
+lei-add-watch - watch for new messages and flag changes
+
+=head1 SYNOPSIS
+
+lei add-watch [OPTIONS] LOCATION [LOCATION...]
+
+=head1 DESCRIPTION
+
+Tell lei to watch C<LOCATION> for new messages and flag changes.
+Currently only Maildir locations are supported.
+
+=for comment
+TODO: Document --state?  Believe valid values are pause, import-ro,
+
+=head1 CONTACT
+
+Feedback welcome via plain-text mail to L<mailto:meta@public-inbox.org>
+
+The mail archives are hosted at L<https://public-inbox.org/meta/> and
+L<http://4uok3hntl7oi7b4uf4rtfwefqeexfzil2w6kgk2jn5z2f764irre7byd.onion/meta/>
+
+=head1 COPYRIGHT
+
+Copyright 2021 all contributors L<mailto:meta@public-inbox.org>
+
+License: AGPL-3.0+ L<https://www.gnu.org/licenses/agpl-3.0.txt>
+
+
+=head1 SEE ALSO
+
+L<lei-ls-watch(1)>, L<lei-rm-watch(1)>
diff --git a/Documentation/lei-forget-mail-sync.pod b/Documentation/lei-forget-mail-sync.pod
new file mode 100644
index 00000000..e70b4d33
--- /dev/null
+++ b/Documentation/lei-forget-mail-sync.pod
@@ -0,0 +1,31 @@
+=head1 NAME
+
+lei-forget-mail-sync - forget sync information for a mail folder
+
+=head1 SYNOPSIS
+
+lei forget-mail-sync [OPTIONS] LOCATION [LOCATION...]
+
+=head1 DESCRIPTION
+
+Forget synchronization information for C<LOCATION>, an IMAP or Maildir
+folder.  Note that this won't delete any messages stored in Git,
+leaving C<lei-index(1)> users with dangling references.
+
+=head1 CONTACT
+
+Feedback welcome via plain-text mail to L<mailto:meta@public-inbox.org>
+
+The mail archives are hosted at L<https://public-inbox.org/meta/> and
+L<http://4uok3hntl7oi7b4uf4rtfwefqeexfzil2w6kgk2jn5z2f764irre7byd.onion/meta/>
+
+=head1 COPYRIGHT
+
+Copyright 2021 all contributors L<mailto:meta@public-inbox.org>
+
+License: AGPL-3.0+ L<https://www.gnu.org/licenses/agpl-3.0.txt>
+
+
+=head1 SEE ALSO
+
+L<lei-ls-mail-sync(1)>, L<lei-index(1)>
diff --git a/Documentation/lei-inspect.pod b/Documentation/lei-inspect.pod
new file mode 100644
index 00000000..19dd8ab5
--- /dev/null
+++ b/Documentation/lei-inspect.pod
@@ -0,0 +1,57 @@
+=head1 NAME
+
+lei-inspect - general purpose inspector
+
+=head1 SYNOPSIS
+
+lei inspect [OPTIONS] ITEM [ITEM...]
+
+lei inspect [OPTIONS] (--stdin|-)
+
+=head1 DESCRIPTION
+
+This is a diagnostic command that provides a general purpose inspector
+of various things, including blobs, message IDs, Xapian document IDs,
+and mail sync sources.
+
+=head1 OPTIONS
+
+=over
+
+=item -d DIR
+
+=item --dir=DIR
+
+An inboxdir, extindex topdir, or Xapian shard
+
+=item --pretty
+
+Pretty print output.  If stdout is opened to a tty, C<--pretty> is
+enabled by default.
+
+=item -
+
+=item --stdin
+
+Read message from stdin.  This is implicit if no arguments are given
+and stdin is a pipe or regular file.
+
+=back
+
+=head1 CONTACT
+
+Feedback welcome via plain-text mail to L<mailto:meta@public-inbox.org>
+
+The mail archives are hosted at L<https://public-inbox.org/meta/> and
+L<http://4uok3hntl7oi7b4uf4rtfwefqeexfzil2w6kgk2jn5z2f764irre7byd.onion/meta/>
+
+=head1 COPYRIGHT
+
+Copyright 2021 all contributors L<mailto:meta@public-inbox.org>
+
+License: AGPL-3.0+ L<https://www.gnu.org/licenses/agpl-3.0.txt>
+
+
+=head1 SEE ALSO
+
+L<lei-mail-diff(1)>
diff --git a/Documentation/lei-ls-mail-source.pod b/Documentation/lei-ls-mail-source.pod
new file mode 100644
index 00000000..926bbe2c
--- /dev/null
+++ b/Documentation/lei-ls-mail-source.pod
@@ -0,0 +1,58 @@
+=head1 NAME
+
+lei-ls-mail-source - list IMAP or NNTP mail source folders
+
+=head1 SYNOPSIS
+
+lei ls-mail-source [OPTIONS] URL
+
+=head1 DESCRIPTION
+
+List information about the IMAP or NNTP mail source at C<URL>.
+
+=head1 OPTIONS
+
+=over
+
+=item -z
+
+=item -0
+
+Use C<\0> (NUL) instead of newline (CR) to delimit lines.
+
+=item -l
+
+Format output as JSON and include more information.
+
+=item --pretty
+
+Pretty print JSON output.  If stdout is opened to a tty, C<--pretty>
+is enabled by default.
+
+=item --ascii
+
+Escape non-ASCII characters.
+
+=item --url
+
+Show full URL of newsgroup or IMAP folder.
+
+=back
+
+=head1 CONTACT
+
+Feedback welcome via plain-text mail to L<mailto:meta@public-inbox.org>
+
+The mail archives are hosted at L<https://public-inbox.org/meta/> and
+L<http://4uok3hntl7oi7b4uf4rtfwefqeexfzil2w6kgk2jn5z2f764irre7byd.onion/meta/>
+
+=head1 COPYRIGHT
+
+Copyright 2021 all contributors L<mailto:meta@public-inbox.org>
+
+License: AGPL-3.0+ L<https://www.gnu.org/licenses/agpl-3.0.txt>
+
+
+=head1 SEE ALSO
+
+L<lei-import(1)>
diff --git a/Documentation/lei-ls-mail-sync.pod b/Documentation/lei-ls-mail-sync.pod
index 86aede40..883eeead 100644
--- a/Documentation/lei-ls-mail-sync.pod
+++ b/Documentation/lei-ls-mail-sync.pod
@@ -52,4 +52,4 @@ License: AGPL-3.0+ L<https://www.gnu.org/licenses/agpl-3.0.txt>
 
 =head1 SEE ALSO
 
-L<lei-q(1)>, L<lei-up(1)>
+L<lei-refresh-mail-sync(1)>, L<lei-export-kw(1)>
diff --git a/Documentation/lei-ls-watch.pod b/Documentation/lei-ls-watch.pod
new file mode 100644
index 00000000..b1681ee4
--- /dev/null
+++ b/Documentation/lei-ls-watch.pod
@@ -0,0 +1,29 @@
+=head1 NAME
+
+lei-ls-watch - list active watches
+
+=head1 SYNOPSIS
+
+lei ls-watch
+
+=head1 DESCRIPTION
+
+List locations that lei is configured to watch.
+
+=head1 CONTACT
+
+Feedback welcome via plain-text mail to L<mailto:meta@public-inbox.org>
+
+The mail archives are hosted at L<https://public-inbox.org/meta/> and
+L<http://4uok3hntl7oi7b4uf4rtfwefqeexfzil2w6kgk2jn5z2f764irre7byd.onion/meta/>
+
+=head1 COPYRIGHT
+
+Copyright 2021 all contributors L<mailto:meta@public-inbox.org>
+
+License: AGPL-3.0+ L<https://www.gnu.org/licenses/agpl-3.0.txt>
+
+
+=head1 SEE ALSO
+
+L<lei-add-watch(1)>, L<lei-rm-watch(1)>
diff --git a/Documentation/lei-mail-diff.pod b/Documentation/lei-mail-diff.pod
new file mode 100644
index 00000000..96e49a8b
--- /dev/null
+++ b/Documentation/lei-mail-diff.pod
@@ -0,0 +1,33 @@
+=head1 NAME
+
+lei-mail-diff - diff the contents of emails
+
+=head1 SYNOPSIS
+
+
+lei mail-diff [OPTIONS] LOCATION
+
+lei mail-diff [OPTIONS] (--stdin|-)
+
+=head1 DESCRIPTION
+
+This is a diagnostic command that's useful for finding deduplication
+bugs.
+
+=head1 CONTACT
+
+Feedback welcome via plain-text mail to L<mailto:meta@public-inbox.org>
+
+The mail archives are hosted at L<https://public-inbox.org/meta/> and
+L<http://4uok3hntl7oi7b4uf4rtfwefqeexfzil2w6kgk2jn5z2f764irre7byd.onion/meta/>
+
+=head1 COPYRIGHT
+
+Copyright 2021 all contributors L<mailto:meta@public-inbox.org>
+
+License: AGPL-3.0+ L<https://www.gnu.org/licenses/agpl-3.0.txt>
+
+
+=head1 SEE ALSO
+
+L<lei-inspect(1)>
diff --git a/Documentation/lei-refresh-mail-sync.pod b/Documentation/lei-refresh-mail-sync.pod
index 92ca9044..65150ae3 100644
--- a/Documentation/lei-refresh-mail-sync.pod
+++ b/Documentation/lei-refresh-mail-sync.pod
@@ -54,4 +54,4 @@ License: AGPL-3.0+ L<https://www.gnu.org/licenses/agpl-3.0.txt>
 
 =head1 SEE ALSO
 
-L<lei-index(1)>, L<lei-export-kw(1)>
+L<lei-index(1)>, L<lei-export-kw(1)>, L<lei-ls-mail-sync(1)>
diff --git a/Documentation/lei-rm-watch.pod b/Documentation/lei-rm-watch.pod
new file mode 100644
index 00000000..711d7dc4
--- /dev/null
+++ b/Documentation/lei-rm-watch.pod
@@ -0,0 +1,30 @@
+=head1 NAME
+
+lei-rm-watch - stop watching locations
+
+=head1 SYNOPSIS
+
+lei rm-watch [OPTIONS] LOCATION [LOCATION...]
+
+=head1 DESCRIPTION
+
+Tell lei to stop watching C<LOCATION> for new messages and flag
+changes.  Currently only Maildir locations are supported.
+
+=head1 CONTACT
+
+Feedback welcome via plain-text mail to L<mailto:meta@public-inbox.org>
+
+The mail archives are hosted at L<https://public-inbox.org/meta/> and
+L<http://4uok3hntl7oi7b4uf4rtfwefqeexfzil2w6kgk2jn5z2f764irre7byd.onion/meta/>
+
+=head1 COPYRIGHT
+
+Copyright 2021 all contributors L<mailto:meta@public-inbox.org>
+
+License: AGPL-3.0+ L<https://www.gnu.org/licenses/agpl-3.0.txt>
+
+
+=head1 SEE ALSO
+
+L<lei-add-watch(1)>, L<lei-ls-watch(1)>
diff --git a/Documentation/lei.pod b/Documentation/lei.pod
index 63d5ee69..24a585da 100644
--- a/Documentation/lei.pod
+++ b/Documentation/lei.pod
@@ -96,6 +96,8 @@ Other subcommands include
 
 =over
 
+=item * L<lei-add-watch(1)>
+
 =item * L<lei-config(1)>
 
 =item * L<lei-convert(1)>
@@ -104,10 +106,22 @@ Other subcommands include
 
 =item * L<lei-daemon-pid(1)>
 
+=item * lei-forget-mail-sync(1)
+
+=item * L<lei-mail-diff(1)>
+
+=item * L<lei-inspect(1)>
+
 =item * L<lei-ls-label(1)>
 
+=item * L<lei-ls-mail-source(1)>
+
 =item * L<lei-ls-mail-sync(1)>
 
+=item * L<lei-ls-watch(1)>
+
+=item * L<lei-rm-watch(1)>
+
 =back
 
 =head1 FILES
diff --git a/Documentation/txt2pre b/Documentation/txt2pre
index bc94d404..fb07579a 100755
--- a/Documentation/txt2pre
+++ b/Documentation/txt2pre
@@ -12,6 +12,7 @@ use PublicInbox::Hval qw(ascii_html);
 my %xurls;
 for (qw[lei(1)
 	lei-add-external(1)
+	lei-add-watch(1)
 	lei-blob(1)
 	lei-config(1)
 	lei-convert(1)
@@ -19,20 +20,26 @@ for (qw[lei(1)
 	lei-daemon-pid(1)
 	lei-edit-search(1)
 	lei-forget-external(1)
+	lei-forget-mail-sync(1)
 	lei-forget-search(1)
 	lei-import(1)
 	lei-index(1)
 	lei-init(1)
+	lei-inspect(1)
 	lei-lcat(1)
 	lei-ls-external(1)
 	lei-ls-label(1)
+	lei-ls-mail-source(1)
 	lei-ls-mail-sync(1)
 	lei-ls-search(1)
+	lei-ls-watch(1)
+	lei-mail-diff(1)
 	lei-overview(7)
 	lei-p2q(1)
 	lei-q(1)
 	lei-rediff(1)
 	lei-rm(1)
+	lei-rm-watch(1)
 	lei-security(7)
 	lei-store-format(5)
 	lei-tag(1)
diff --git a/MANIFEST b/MANIFEST
index b89513d5..c4cc6e33 100644
--- a/MANIFEST
+++ b/MANIFEST
@@ -22,6 +22,7 @@ Documentation/flow.txt
 Documentation/hosted.txt
 Documentation/include.mk
 Documentation/lei-add-external.pod
+Documentation/lei-add-watch.pod
 Documentation/lei-blob.pod
 Documentation/lei-config.pod
 Documentation/lei-convert.pod
@@ -31,15 +32,20 @@ Documentation/lei-daemon.pod
 Documentation/lei-edit-search.pod
 Documentation/lei-export-kw.pod
 Documentation/lei-forget-external.pod
+Documentation/lei-forget-mail-sync.pod
 Documentation/lei-forget-search.pod
 Documentation/lei-import.pod
 Documentation/lei-index.pod
 Documentation/lei-init.pod
+Documentation/lei-inspect.pod
 Documentation/lei-lcat.pod
 Documentation/lei-ls-external.pod
 Documentation/lei-ls-label.pod
+Documentation/lei-ls-mail-source.pod
 Documentation/lei-ls-mail-sync.pod
 Documentation/lei-ls-search.pod
+Documentation/lei-ls-watch.pod
+Documentation/lei-mail-diff.pod
 Documentation/lei-mail-formats.pod
 Documentation/lei-overview.pod
 Documentation/lei-p2q.pod
@@ -47,6 +53,7 @@ Documentation/lei-q.pod
 Documentation/lei-rediff.pod
 Documentation/lei-refresh-mail-sync.pod
 Documentation/lei-rm.pod
+Documentation/lei-rm-watch.pod
 Documentation/lei-security.pod
 Documentation/lei-store-format.pod
 Documentation/lei-tag.pod
diff --git a/Makefile.PL b/Makefile.PL
index 22a58bee..348a343d 100644
--- a/Makefile.PL
+++ b/Makefile.PL
@@ -45,13 +45,13 @@ $v->{-m1} = [ map {
 		}
 	} @EXE_FILES,
 	qw(
-	lei-add-external lei-blob lei-config lei-convert
+	lei-add-external lei-add-watch lei-blob lei-config lei-convert
 	lei-daemon-kill lei-daemon-pid lei-edit-search lei-export-kw
-	lei-forget-external lei-forget-search
-	lei-import lei-index lei-init lei-lcat
-	lei-ls-external lei-ls-label lei-ls-mail-sync
-	lei-ls-search lei-p2q lei-q
-	lei-rediff lei-refresh-mail-sync lei-rm lei-tag
+	lei-forget-external lei-forget-mail-sync lei-forget-search
+	lei-import lei-index lei-init lei-inspect lei-lcat
+	lei-ls-external lei-ls-label lei-ls-mail-source lei-ls-mail-sync
+	lei-ls-search lei-ls-watch lei-mail-diff lei-p2q lei-q
+	lei-rediff lei-refresh-mail-sync lei-rm lei-rm-watch lei-tag
 	lei-up)];
 $v->{-m5} = [ qw(public-inbox-config public-inbox-v1-format
 		public-inbox-v2-format public-inbox-extindex-format
-- 
2.33.0

Re: [PATCH 2/2] doc: lei: add manpages for remaining commands

[Eric Wong]

Kyle Meyer <kyle@kyleam.com> wrote:
> At this point all of the current lei commands, aside from -help and
> -sucks, should be covered.

Thanks, pushed as commit 9d72cc3f876e3d2bd1ecb2fc0f33c43a9a72b933

> diff --git a/MANIFEST b/MANIFEST
> index b89513d5..c4cc6e33 100644
> --- a/MANIFEST
> +++ b/MANIFEST

> @@ -47,6 +53,7 @@ Documentation/lei-q.pod
>  Documentation/lei-rediff.pod
>  Documentation/lei-refresh-mail-sync.pod
>  Documentation/lei-rm.pod
> +Documentation/lei-rm-watch.pod
>  Documentation/lei-security.pod
>  Documentation/lei-store-format.pod
>  Documentation/lei-tag.pod

Curious, was that from `git ls-files >MANIFEST' ?

Because that puts lei-rm.pod after lei-rm-watch.pod on different
systems I tested.  I don't see git using LC_COLLATE anywhere,
but locale(1) gives:

FreeBSD:
	LANG=
	LC_CTYPE="C"
	LC_COLLATE="C"
	LC_TIME="C"
	LC_NUMERIC="C"
	LC_MONETARY="C"
	LC_MESSAGES="C"
	LC_ALL=

Debian:
	LANG=en_US.UTF-8
	LANGUAGE=
	LC_CTYPE="en_US.UTF-8"
	LC_NUMERIC="en_US.UTF-8"
	LC_TIME="en_US.UTF-8"
	LC_COLLATE="en_US.UTF-8"
	LC_MONETARY="en_US.UTF-8"
	LC_MESSAGES="en_US.UTF-8"
	LC_PAPER="en_US.UTF-8"
	LC_NAME="en_US.UTF-8"
	LC_ADDRESS="en_US.UTF-8"
	LC_TELEPHONE="en_US.UTF-8"
	LC_MEASUREMENT="en_US.UTF-8"
	LC_IDENTIFICATION="en_US.UTF-8"
	LC_ALL=

No big deal, I can just flip and push it; I just don't want a
reproducibility issue popping up.

Re: [PATCH 2/2] doc: lei: add manpages for remaining commands

[Kyle Meyer]

Eric Wong writes:

>> @@ -47,6 +53,7 @@ Documentation/lei-q.pod
>>  Documentation/lei-rediff.pod
>>  Documentation/lei-refresh-mail-sync.pod
>>  Documentation/lei-rm.pod
>> +Documentation/lei-rm-watch.pod
>>  Documentation/lei-security.pod
>>  Documentation/lei-store-format.pod
>>  Documentation/lei-tag.pod
>
> Curious, was that from `git ls-files >MANIFEST' ?

No, it wasn't.  I hadn't realized that the content of MANIFEST
corresponded directly to git-ls-files output, but poking around I see
there is even a check-manifest make target [*].  Doh.

So, I manually added this one, like the others I added.  I think my past
changes to this file have ended up in the expected order because I tend
to run Emacs's sort-lines to check my manual placement, but I must not
have done it here because sort-lines agrees with ls-files.

[...]
> No big deal, I can just flip and push it; I just don't want a
> reproducibility issue popping up.

Thanks, sorry about the mix up.


[*] I did run `make check' before sending, and it looks like that also
    checks MANIFEST _after_ running the test suite.  That didn't help me
    catch the MANIFEST sorting issue in this case because the test suite
    is failing on my end.

    I believe these failures are recent and was planning on looking into
    them today, either sending a patch or just reporting, depending on
    whether I could figure out a fix.  Anyway, here they are:

      t/psgi_multipart_not.t .......
      ok 1 - use HTTP::Request::Common;
      ok 2 - use Plack::Test;
      ok 3 - use PublicInbox::WWW;
      not ok 4 - /v2test/?q=%22ain't what it used to be%22&x=t
      not ok 5 - /v2test/?q=%22ain't what it used to be%22&x=t warns
      Failed 2/5 subtests

Re: [PATCH 2/2] doc: lei: add manpages for remaining commands

[Kyle Meyer]

Kyle Meyer writes:

>       t/psgi_multipart_not.t .......
>       ok 1 - use HTTP::Request::Common;
>       ok 2 - use Plack::Test;
>       ok 3 - use PublicInbox::WWW;
>       not ok 4 - /v2test/?q=%22ain't what it used to be%22&x=t
>       not ok 5 - /v2test/?q=%22ain't what it used to be%22&x=t warns
>       Failed 2/5 subtests

Hmm, trying a few times on 9ee1798e (2021-10-16), I'm not having any
luck triggering it again.  All tests are now passing for me.

Re: [PATCH 2/2] doc: lei: add manpages for remaining commands

[Eric Wong]

Kyle Meyer <kyle@kyleam.com> wrote:
> Thanks, sorry about the mix up.

No worries, will push a regen.

> [*] I did run `make check' before sending, and it looks like that also
>     checks MANIFEST _after_ running the test suite.  That didn't help me
>     catch the MANIFEST sorting issue in this case because the test suite
>     is failing on my end.
> 
>     I believe these failures are recent and was planning on looking into
>     them today, either sending a patch or just reporting, depending on
>     whether I could figure out a fix.  Anyway, here they are:
> 
>       t/psgi_multipart_not.t .......
>       ok 1 - use HTTP::Request::Common;
>       ok 2 - use Plack::Test;
>       ok 3 - use PublicInbox::WWW;
>       not ok 4 - /v2test/?q=%22ain't what it used to be%22&x=t
>       not ok 5 - /v2test/?q=%22ain't what it used to be%22&x=t warns
>       Failed 2/5 subtests

The failure is probably caused by
00d5dff2cce9d2c9 (eml: avoid Encode 2.87..3.12 leak, 2021-10-13)
but I can't reproduce it across CentOS 7.x, FreeBSD 11.x, nor
Debian 10 & 11.

Which versions of Encode and Perl are you using?

At least in Debian, libencode-perl is available as a separate package
but it's also part of libperl5.xx (possibly w/ a different version);
only the latter is required for us, but two packages offering
the same thing gets confusing :/

I use: perl -MEncode -E 'say $Encode::VERSION'
to determine which gets loaded.

Re: [PATCH 2/2] doc: lei: add manpages for remaining commands

[Kyle Meyer]

Eric Wong writes:

> The failure is probably caused by
> 00d5dff2cce9d2c9 (eml: avoid Encode 2.87..3.12 leak, 2021-10-13)
> but I can't reproduce it across CentOS 7.x, FreeBSD 11.x, nor
> Debian 10 & 11.

Hmm, yeah, as mentioned in my other reply, I'm now not having any luck
triggering this either.

> Which versions of Encode and Perl are you using?

Perl v5.32.1 from Debian 11 and...

> At least in Debian, libencode-perl is available as a separate package
> but it's also part of libperl5.xx (possibly w/ a different version);
> only the latter is required for us, but two packages offering
> the same thing gets confusing :/
>
> I use: perl -MEncode -E 'say $Encode::VERSION'
> to determine which gets loaded.

... it looks like I'm using the on that's ships with libperl5:

  $ perl -MEncode -E 'say $Encode::VERSION'
  3.06

  $ apt-cache policy libencode-perl
  libencode-perl:
    Installed: (none)
    Candidate: 3.08-1+deb11u1
    Version table:
       3.08-1+deb11u1 500
          500 http://ftp.us.debian.org/debian bullseye/main amd64 Packages
          500 http://security.debian.org/debian-security bullseye-security/main amd64 Packages