Re: [RFC] Clean way to disable pager

Matthieu Moy wrote:

David Kastrup [off-list ref] writes:

quoted

Adam Roben [off-list ref] writes:

quoted

On Aug 19, 2007, at 12:44 PM, Matthieu Moy wrote:

quoted

+    [-p|--paginate] [--no-pager]

  I think that [-p|--[no-]paginate] would be more consistent with the
way negatable options are normally specified.

Disregard previously existing code, I'd vote for using an option set
like the following:

--pager
--pager=less
--pager=cat
--no-pager

"--paginate" is rather artificial in contrast.

ACK. I thought of naming the option --no-paginate, but I don't think
that good english. --dont-paginate would be good english, but doesn't
match the --[no-]whatever scheme.

Perhaps using David's proposal, and keeping the --paginate as a
backward compatibility, deprecated switch is the way to go.

I like it too. ACK.

-- 
Jakub Narebski
Warsaw, Poland
ShadeHawk on #git