Re: [PATCH v2] Clarify Inline::C dependency (optional on Linux, required elsewhere)

[Eric Wong <e@80x24.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