[PATCH] pager: teeny tiny documentation patch
Kevin Bullock
kbullock+mercurial at ringworld.org
Sat Oct 30 23:36:18 CDT 2010
On 30 Oct 2010, at 2:20 AM, Eduard-Cristian Stefan wrote:
>>> pager: make example of setting pager less silly
> ...
> - pager = LESS='FSRX' less
> + pager = less -FRSX
>
> AFAIK those are two valid ways to pass options to less:
> one sets an environment variable, and the other
> is passing them on the command line.
>
> Maybe both should appear in the documentation,
> as neither of them is silly.
I say let less(1) document how to pass options to less. The environment variable is provided mostly to allow setting the default options login-wide. Setting the env var as a prefix to a single invocation (especially with unnecessary single quotes) is, in my estimation, silly. But thanks for reviewing the patch.
> Have a nice day,
> Eduard
pacem in terris / mir / shanti / salaam / heiwa
Kevin R. Bullock
More information about the Mercurial-devel
mailing list