D6709: config: add --registered flag to show all known configs

navaneeth.suresh (Navaneeth Suresh) phabricator at mercurial-scm.org
Thu Aug 8 12:08:45 EDT 2019


navaneeth.suresh added a comment.


  @pulkit I couldn't figure out why the devel-warnings are coming. It's being shown for the following 
  items in `configitems.py`. IIUC, it should be visible when we are trying to read a config item which is
  not in the registered items.  But, we are iterating over registered items only.
  
    coreconfigitem('help', br'hidden-command\..*',
        default=False,
        generic=True,
    )
    coreconfigitem('help', br'hidden-topic\..*',
        default=False,
        generic=True,
    )
    coreconfigitem('hostsecurity', '.*:fingerprints$',
        default=list,
        generic=True,
    )
    coreconfigitem('hostsecurity', '.*:verifycertsfile$',
        default=None,
        generic=True,
    )
  
  Also, why would we want `walkregisteredconfig()` to return `section, name, value, defaultvalue` ?
  If we do that, then it would require modification in `ui.walkregisteredconfig()` and `commands.config()`.
  
  @av6 If you have a strong opinion on replacing `True/False` with `yes/no`, I can do that.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D6709/new/

REVISION DETAIL
  https://phab.mercurial-scm.org/D6709

To: navaneeth.suresh, #hg-reviewers, av6, marmoute
Cc: pulkit, marmoute, av6, mjpieters, mercurial-devel


More information about the Mercurial-devel mailing list