From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <e@80x24.org>
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on dcvr.yhbt.net
X-Spam-Level: 
X-Spam-ASN:  
X-Spam-Status: No, score=-4.2 required=3.0 tests=ALL_TRUSTED,AWL,BAYES_00,
	DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,
	T_SCC_BODY_TEXT_LINE shortcircuit=no autolearn=ham autolearn_force=no
	version=3.4.6
Received: from localhost (dcvr.yhbt.net [127.0.0.1])
	by dcvr.yhbt.net (Postfix) with ESMTP id 465D01F44D;
	Sat, 16 Mar 2024 21:34:04 +0000 (UTC)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=80x24.org;
	s=selector1; t=1710624844;
	bh=ZU8oXH1pUIX7rhHsz0XQGt3dA5MJE171tnmVlWVu3Xs=;
	h=Date:From:To:Cc:Subject:References:In-Reply-To:From;
	b=YfQBSxjWtgZ11GzkpMQ+BkYQgfTNxk62FInm5zoPMR0pcGTb5iwt99DpVFbiHvcPA
	 I9AuhKSF3GH1XB2ejL4xdomwUaQORwItmA2hK2iD1tYqC0CwNIEkdCQwz6miKfB+Aq
	 U5rwXhf5hUsHuPKvoPg56O0Qal846pdW+ZTIuPEU=
Date: Sat, 16 Mar 2024 21:27:56 +0000
From: Eric Wong <e@80x24.org>
To: =?utf-8?B?xaB0xJtww6FuIE7Em21lYw==?= <stepnem@smrk.net>
Cc: meta@public-inbox.org
Subject: Re: [PATCH (RFC) 2/2] INSTALL: try to be less confusing about
 optional modules
Message-ID: <20240316212756.M816616@dcvr>
References: <20240315143049.1590728-1-stepnem@smrk.net>
 <20240315143049.1590728-2-stepnem@smrk.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20240315143049.1590728-2-stepnem@smrk.net>
List-Id: <meta.public-inbox.org>

Štěpán Němec <stepnem@smrk.net> 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,