Štěpán Němec <[email protected]> wrote:
> The difference between the "numerous optional modules"
> section (containing only two modules) and the "everything
> else optional" section was unclear (to me, at least).
> Just put both under a single heading.

<snip>

> +++ b/INSTALL
> @@ -58,7 +58,8 @@ Where "deb" indicates package names for Debian-derived 
> distributions,
>  "pkgin" for NetBSD, "apk" for Alpine Linux and "rpm" is for RPM-based
>  distributions (only known to work on Fedora).
>  
> -Numerous optional modules are likely to be useful as well:
> +Numerous optional modules might be useful as well, depending
> +on your use case and preferences:
>  

OK, I like the first change.

>  - DBD::SQLite                      deb: libdbd-sqlite3-perl
>                                     pkg: p5-DBD-SQLite
> @@ -71,8 +72,6 @@ Numerous optional modules are likely to be useful as well:
>                                     rpm: perl-Search-Xapian
>                                     (required for lei; HTTP and IMAP search)
>  
> -Every effort has been to make everything else optional:
> -

I consider DBD::SQLite and Xapian significantly more important
than the rest, so I favor keeping the above line.

>  - Plack                            deb: libplack-perl
>                                     pkg: p5-Plack
>                                     rpm: perl-Plack, perl-Plack-Test,

Reply via email to