Re: [PATCH 1/3] doc-diff: fix non-portable 'man' invocation
- From
Jeff King <peff@peff.net>
- Date
- Aug 31, 2018, 19:49 UTC
- Message-ID
- <20180831194927.GB5120@sigill.intra.peff.net>
- In-Reply-To
- <20180831063318.33373-2-sunshine@sunshineco.com>
On Fri, Aug 31, 2018 at 02:33:16AM -0400, Eric Sunshine wrote:
Show 11 quoted lines
> doc-diff invokes 'man' with the -l option to force "local" mode, > however, neither MacOS nor FreeBSD recognize this option. On those > platforms, if the argument to 'man' contains a slash, it is > automatically interpreted as a file specification, so a "local"-like > mode is not needed. And, it turns out, 'man' which does support -l > falls back to enabling -l automatically if it can't otherwise find a > manual entry corresponding to the argument. Since doc-diff always > passes an absolute path of the nroff source file to 'man', the -l > option kicks in anyhow, despite not being specified explicitly. > Therefore, make the invocation portable to the various platforms by > simply dropping -l.
Neat. Today I learned.
Confirmed that this works just fine without "-l" on my system (and that "./foo.1" is an easy alternative to "man -l" on other systems).
-Peff