On Thu, May 01, 2008 at 11:56:15PM -0700, Jakub Narebski wrote: > > Here is a quick and dirty patch to do that. It should probably be split > > into two (there is a big code movement of the commands array), and it > > needs documentation and tests. But I'm going to sleep for now. > > You should then accept pager.core (or pager.ui) as alias to existing > core.pager configuration variable... well, perhaps you did that... Why? I am claiming the pager.* namespace for turning the pager off and on for specific commands, which neither of those do (the exception is the historic pager.color, which would still work). If you want to argue that all pager options should be consolidated under pager.*, I think that is a totally separate issue (and one which my patch argues against, since it could lead to name collisions). -Peff -- 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
