Patchwork [1,of,2] pager: improve help for --pager=

login
register
mail settings
Submitter timeless@mozdev.org
Date Nov. 30, 2015, 7:09 p.m.
Message ID <a92289f0420c849b40f8.1448910594@waste.org>
Download mbox | patch
Permalink /patch/11682/
State Accepted
Headers show

Comments

timeless@mozdev.org - Nov. 30, 2015, 7:09 p.m.
# HG changeset patch
# User timeless <timeless@mozdev.org>
# Date 1448909253 0
#      Mon Nov 30 18:47:33 2015 +0000
# Node ID a92289f0420c849b40f86904d935c7598a49cfa4
# Parent  61fbf5dc12b23e7a2a30cf04ebd9f096c42a1f61
pager: improve help for --pager=

try to clarify how to enable/disable the pager
Sean Farley - Nov. 30, 2015, 7:56 p.m.
timeless <timeless@mozdev.org> writes:

> # HG changeset patch
> # User timeless <timeless@mozdev.org>
> # Date 1448909253 0
> #      Mon Nov 30 18:47:33 2015 +0000
> # Node ID a92289f0420c849b40f86904d935c7598a49cfa4
> # Parent  61fbf5dc12b23e7a2a30cf04ebd9f096c42a1f61
> pager: improve help for --pager=
>
> try to clarify how to enable/disable the pager

Besides 'try' not being capitalized, these patches seems fine to me.
Pierre-Yves David - Nov. 30, 2015, 8:55 p.m.
On 11/30/2015 11:09 AM, timeless wrote:
> # HG changeset patch
> # User timeless <timeless@mozdev.org>
> # Date 1448909253 0
> #      Mon Nov 30 18:47:33 2015 +0000
> # Node ID a92289f0420c849b40f86904d935c7598a49cfa4
> # Parent  61fbf5dc12b23e7a2a30cf04ebd9f096c42a1f61
> pager: improve help for --pager=

I've pushed patch 1,

Thanks (and extra tahanks to Sean for proof reading it).

Patch

diff --git a/hgext/pager.py b/hgext/pager.py
--- a/hgext/pager.py
+++ b/hgext/pager.py
@@ -49,9 +49,13 @@ 
 To ignore global commands like :hg:`version` or :hg:`help`, you have
 to specify them in your user configuration file.
 
-The --pager=... option can also be used to control when the pager is
-used. Use a boolean value like yes, no, on, off, or use auto for
-normal behavior.
+To control whether the pager is used at all for an individual command,
+you can use --pager=<value>::
+
+  - use as needed: `auto`.
+  - require the pager: `yes` or `on`.
+  - suppress the pager: `no` or `off` (any unrecognized value
+  will also work).
 
 '''