[PATCH RFC] ui: introduce sysdefault section for pager and editor configuration

Sean Farley sean at farley.io
Thu Mar 9 21:49:18 EST 2017


Kevin Bullock <kbullock+mercurial at ringworld.org> writes:

>> On Mar 8, 2017, at 18:06, Jun Wu <quark at fb.com> wrote:
>> 
>> A "sysdefault" section sounds a bit weird. Usually files under "/etc" are
>> defining "system defaults". A new section also makes it harder to see what
>> configs they are overriding.
>> 
>> How about appending ":sysdefault" to normal configs? The idea was also
>> mentioned by Yuya at [1] and we use them in "[paths]":
>> 
>>  [ui]
>>  pager:sysdefault = sensible-pager
>>  editor:sysdefault = sensible-editor
>> 
>> This makes it easier to see what configs they are overriding, and makes the
>> section less crowded if we want the same thing for other sections.
>
> I like that idea.

I guess I don't follow since I think of things in /etc as the system
defaults to begin with. Are we having a namespace clash with users
having 'foo=BAR' set so that setting 'foo' in /etc won't override that?


More information about the Mercurial-devel mailing list