[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