Re: [PATCH/DONOTAPPLY 0/4] first steps towards pager.foo.{command,enable}

2017-11-07 Thread Martin Ågren
On 6 November 2017 at 11:48, Jeff King wrote: > On Sun, Nov 05, 2017 at 12:58:18PM +0100, Martin Ågren wrote: >> In particular, they do not teach `--paginate` to use the pager >> configured by `pager.foo.command`. It is already now possible to use >> `pager.foo` to say "I don't want you to page, b

Re: [PATCH/DONOTAPPLY 0/4] first steps towards pager.foo.{command,enable}

2017-11-06 Thread Jeff King
On Sun, Nov 05, 2017 at 12:58:18PM +0100, Martin Ågren wrote: > On 4 November 2017 at 10:28, Jeff King wrote: > > - the pager. config is mis-designed, because our config keys > >cannot represent all possible command names (e.g., case folding and > >illegal characters). This should be pag

[PATCH/DONOTAPPLY 0/4] first steps towards pager.foo.{command,enable}

2017-11-05 Thread Martin Ågren
I'm not posting this for inclusion (yet), but because I read this: On 4 November 2017 at 10:28, Jeff King wrote: > - the pager. config is mis-designed, because our config keys >cannot represent all possible command names (e.g., case folding and >illegal characters). This should be pager.