Matthieu Moy wrote: > David Kastrup <dak@xxxxxxx> writes: >> Adam Roben <aroben@xxxxxxxxx> writes: >>> On Aug 19, 2007, at 12:44 PM, Matthieu Moy wrote: >>> >>>> + [-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 - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html