Re: [PATCH 1/3] doc: fix some typos and grammar

[Eric Wong <e@80x24.org>]

Štěpán Němec <stepnem@smrk.net> wrote:
>  Documentation/RelNotes/v1.9.0.eml        | 2 +-
>  Documentation/RelNotes/v2.0.0.wip        | 6 +++---
>  Documentation/lei-config.pod             | 8 ++++----
>  Documentation/lei-daemon-kill.pod        | 2 +-
>  Documentation/lei-inspect.pod            | 4 ++--
>  Documentation/lei-lcat.pod               | 2 +-
>  Documentation/lei-ls-mail-source.pod     | 2 +-
>  Documentation/lei-ls-search.pod          | 2 +-
>  Documentation/lei-mail-sync-overview.pod | 2 +-
>  Documentation/lei-q.pod                  | 4 ++--
>  Documentation/lei-security.pod           | 8 ++++----
>  Documentation/lei-store-format.pod       | 2 +-
>  Documentation/lei-up.pod                 | 4 ++--

Thanks, changes to those files look fine (but I'm no English
expert, either).  My brain does jumble words often :x

> diff --git a/TODO b/TODO
> index de628e2e310a..36a7f0cf441a 100644
> --- a/TODO
> +++ b/TODO
> @@ -26,7 +26,7 @@ all need to be considered for everything we introduce.)
>    archive locations to avoid SPOF.
>  
>  * optional Cache::FastMmap support so production deployments won't
> -  need Varnish (Varnish doesn't protect NNTP nor IMAP, either)
> +  need Varnish (Varnish doesn't protect NNTP or IMAP, either)

I'm not sure about this change, I seem to remember `nor' being
more suited for negatives.  Any citations or comments from
others reading this?

>  * dogfood and take advantage of new kernel APIs (while maintaining
>    portability to older Linux, free BSDs and maybe Hurd).
> diff --git a/install/README b/install/README
> index 981e014b9959..e1b5558e0b87 100644
> --- a/install/README
> +++ b/install/README
> @@ -5,7 +5,7 @@ TL;DR:	sudo ./install/deps.perl all
>  
>  Files in this directory are designed for:
>  
> -* users using from git or tarballs (and not the OS package manager)
> +* users using git repo or tarballs (and not the OS package manager)

I wonder if:

  * users using our code from git or tarballs (and not the OS package manager)

...is better.  I always feel the need to clarify git the software
vs. our code repo vs. our mirror on public-inbox.org/git   *shrug*

> --- a/lib/PublicInbox/XapHelperCxx.pm
> +++ b/lib/PublicInbox/XapHelperCxx.pm
> @@ -36,7 +36,7 @@ sub xap_cfg (@) {
>  	seek($err, 0, SEEK_SET) or die "seek: $!";
>  	$err = do { local $/; <$err> };
>  	die <<EOM;
> -@$cmd failed: Xapian devlopment files missing? (\$?=$?)
> +@$cmd failed: Xapian development files missing? (\$?=$?)

Yup, I also just noticed that on a different branch I'm been working on.