# HG changeset patch # User Raphaël Gomès # Date 1674490874 -3600 # Node ID f0ae403bf704f1f6bb2af15db87b099d31eb80c3 # Parent 5d092194ac37d70f7b38880070e11b126f84c233 configitems: use standard "dynamicdefault" approach in edge case This makes for fewer edge cases, which will help a future patch. diff -r 5d092194ac37 -r f0ae403bf704 mercurial/configitems.py --- a/mercurial/configitems.py Tue Jan 17 14:53:18 2023 +0100 +++ b/mercurial/configitems.py Mon Jan 23 17:21:14 2023 +0100 @@ -1473,22 +1473,10 @@ b'usestore', default=True, ) - - -def _persistent_nodemap_default(): - """compute `use-persistent-nodemap` default value - - The feature is disabled unless a fast implementation is available. - """ - from . import policy - - return policy.importrust('revlog') is not None - - coreconfigitem( b'format', b'use-persistent-nodemap', - default=_persistent_nodemap_default, + default=dynamicdefault, ) coreconfigitem( b'format', diff -r 5d092194ac37 -r f0ae403bf704 mercurial/localrepo.py --- a/mercurial/localrepo.py Tue Jan 17 14:53:18 2023 +0100 +++ b/mercurial/localrepo.py Mon Jan 23 17:21:14 2023 +0100 @@ -58,6 +58,7 @@ obsolete, pathutil, phases, + policy, pushkey, pycompat, rcutil, @@ -3763,7 +3764,11 @@ if ui.configbool(b'format', b'bookmarks-in-store'): requirements.add(requirementsmod.BOOKMARKS_IN_STORE_REQUIREMENT) - if ui.configbool(b'format', b'use-persistent-nodemap'): + # The feature is disabled unless a fast implementation is available. + persistent_nodemap_default = policy.importrust('revlog') is not None + if ui.configbool( + b'format', b'use-persistent-nodemap', persistent_nodemap_default + ): requirements.add(requirementsmod.NODEMAP_REQUIREMENT) # if share-safe is enabled, let's create the new repository with the new