Re: [PATCH 0/7] tag: more fine-grained pager-configuration
- From
- Martin Ågren <martin.agren@gmail.com>
- Date
- Jul 12, 2017, 07:29 UTC
- Message-ID
- <CAN0heSq-HHsW5Cd_VvshNS_qHW3nVz-yzwSfFQin8Dyr2hhksA@mail.gmail.com>
- In-Reply-To
- <xmqqpod752yh.fsf@gitster.mtv.corp.google.com>
On 11 July 2017 at 00:42, Junio C Hamano <gitster@pobox.com> wrote:
Show 6 quoted lines
> Martin Ågren <martin.agren@gmail.com> writes: >> I am not moving all builtins to handling the pager on their own, >> but instead introduce a flag IGNORE_PAGER_CONFIG and use it only >> with the tag builtin. That means there's another flag to reason >> about, but it avoids moving all builtins to handling the paging >> themselves just to make one of them do something more "clever".
...
Show 6 quoted lines
> Even though it is purely internal thing, IGNORE_PAGER_CONFIG > probably is a bit confusion-inducing name. After all, the > subcommand specific configuration is not being ignored---we are > merely delaying our reaction to it---instead of acting on it inside > git.c without giving the subcommand a chance to make a decision, we > are still letting (and we do expect) the subcommand to react to it.
Thank you for your comments. I'm working on v2 with the input from you, Peff and Brandon. I'll make this DELAY_PAGER_CONFIG.
Martin