Štěpán Němec <[email protected]> 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
