[PATCH RFC] ui: add support for a tweakdefaults knob
yuya at tcha.org
Mon Jun 19 10:58:35 EDT 2017
On Mon, 19 Jun 2017 16:04:41 +0200, Pierre-Yves David wrote:
> >> There have been talk about such registry of option for quite some time
> >> (including at the 4.2 central sprint). David Demelier ping me about
> >> this again 10 days ago and found some time to poke at it yesterday.
> >> It turned out I ended up with something that work, I've started
> >> patchbombing it. Once this is in, we can update the tweakdefault flag
> >> to use this.
> >>  as part of his quest for
> >> https://www.mercurial-scm.org/wiki/ConfigConsolidationPlan
> >> 
> >> https://www.mercurial-scm.org/pipermail/mercurial-devel/2017-June/099866.html
> > These patches look a good move, though I think the config registry idea is
> > bloated a bit. Suppose we have tons of config items, I don't think it's good
> > idea to register them per item.
> I'm not sure what is your concerns here can you elaborate? We have about
> 200 known config option in core. That seems a manageable number of object.
Sorry, I just felt it's overengineered. I don't have no particular concern
> As centralized config growth feature it will needs one item for each
> config option anyway (alias, default, documentation, etc...).
> Do you have an alternative idea in mind?
Hmm. If we want these features, perhaps configitem() function would be the
More information about the Mercurial-devel