[PATCH v3] configitems: add alias support in config

Yuya Nishihara yuya at tcha.org
Thu Jul 6 09:18:44 EDT 2017


On Wed, 05 Jul 2017 13:56:55 +0200, David Demelier wrote:
> # HG changeset patch
> # User David Demelier <demelier.david at gmail.com>
> # Date 1499079875 -7200
> #      Mon Jul 03 13:04:35 2017 +0200
> # Node ID 92546f63e0bee8fa71fd46485e0970b335e438e6
> # Parent  5c9ad50fd62fbc2057ef7b44f921f22e7359af32
> configitems: add alias support in config
> 
> Aliases define optional alternatives to existing options. For example the old
> option ui.user was deprecated and replaced by ui.username. With this mechanism,
> it's even possible to create an alias to an option in a different section.
> 
> Add ui.user as alias to ui.username as an example of this concept.
> 
> The old alternates principle in ui.config is removed as it was used only for
> this option.
> 
> Update hgweb methods to use ui._unset.

Looks good, but any reason why hgweb changes are folded into one patch? I
think the previous series was better in the way of how patches were organized.

> --- /dev/null	Thu Jan 01 00:00:00 1970 +0000
> +++ b/tests/test-config-alias.t	Mon Jul 03 13:04:35 2017 +0200

Both this and test-config.t are small. If you don't have a strong reason
to create new test file, please add new test to test-config.t.


More information about the Mercurial-devel mailing list