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.1 required=3.0 tests=ALL_TRUSTED,AWL,BAYES_00,
	DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF 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 0EE071F55F;
	Sat,  2 Sep 2023 20:58:09 +0000 (UTC)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=80x24.org;
	s=selector1; t=1693688289;
	bh=X5oh2/2dvJgIhZNdarwleUgF418eb2fXEq9n8OfC+yc=;
	h=Date:From:To:Cc:Subject:References:In-Reply-To:From;
	b=KtvQMXVJDU+ypdcFG1EHO0xxQ9FxZVkwkqzy49KO9OfOtTcJu1y5KTAGATgK2ltFT
	 +MpDaLIPkiUmlCR9qk/nVrO6siAr24yBX/9+MwCBr+HM6FZPu0UB4icGob4y5GM4BQ
	 heXVmcahgPMEFUZ37pZcem3dWWhylUbSuX135ZFA=
Date: Sat, 2 Sep 2023 20:56:27 +0000
From: Eric Wong <e@80x24.org>
To: =?utf-8?B?xaB0xJtww6FuIE7Em21lYw==?= <stepnem@smrk.net>
Cc: meta@public-inbox.org
Subject: Re: [PATCH v2] Clarify Inline::C dependency (optional on Linux,
 required elsewhere)
Message-ID: <20230902205627.M260262@dcvr>
References: <20230830211849.M275594@dcvr>
 <20230831111152+0200.360729-stepnem@smrk.net>
 <20230831192650+0200.929387-stepnem@smrk.net>
 <20230901110903.M876537@dcvr>
 <20230902125449+0200.22309-stepnem@smrk.net>
 <20230902130741+0200.224663-stepnem@smrk.net>
 <20230902185015.M198704@dcvr>
 <20230902210825+0200.747967-stepnem@smrk.net>
 <20230902194407.M464597@dcvr>
 <20230902224501+0200.863993-stepnem@smrk.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20230902224501+0200.863993-stepnem@smrk.net>
List-Id: <meta.public-inbox.org>

Štěpán Němec <stepnem@smrk.net> wrote:
> On Sat, 02 Sep 2023 19:44:07 +0000
> Eric Wong wrote:
> 
> > Perhaps the per-module "optional, for .." statements should be
> > "only for ...".
> 
> Well, if that's really true (i.e., they're not needed/useful for
> anything else)...
> 
> I guess you see INSTALL as mainly for public-inbox without lei (and
> indeed the first sentence is "This is for folks who want to set up their
> own public-inbox instance."), so from that perspective saying "optional"
> about something that is optional for public-inbox but required for lei
> makes sense, but lei is then mentioned repeatedly in the dependencies
> section, and it seems a safe bet there are much more people using lei
> than there are people setting up public-inbox instances, so I'm not sure
> maintaining that perspective in INSTALL is the best way to help those
> users.

Perhaps a separate "INSTALL-lei" file is warranted.  lei is newer,
and parts of it still somewhat unrefined/experimental IMHO.

> Then again, maybe it's just in my head and I might as well let such
> users speak for themselves if they exist and ever wish to do so.

Thanks for the different perspective.  In my mind, public-inbox-mda
and PublicInbox::WWW were "first" and everything came after.

> So here's just a reroll with your suggestions implemented:
> 
> -- 8< --
> Subject: [PATCH v3] Clarify Inline::C dependency (optional on Linux, required
>  elsewhere)

Thanks, applied as commit 18c6c4e81dca1d81f6ff2bb284554b5fca2ef163