changeset 46208:083438d6f403

upgrade: drop support for old style optimization names Old style optimization names like `redeltaparent` were converted into `re-delta-parent` more than two years ago. The old names were kept around for sometime because of BC reasons. Refer 5608b5a6c3231c4ec771171cc3079142c7672be6. The commit states the map is there for a while and we can drop them as the underlying command is a debug command. Differential Revision: https://phab.mercurial-scm.org/D9614
author Pulkit Goyal <7895pulkit@gmail.com>
date Wed, 16 Dec 2020 12:39:15 +0530
parents e2139e071b5c
children a51d345f1404
files mercurial/debugcommands.py mercurial/upgrade.py tests/test-upgrade-repo.t
diffstat 3 files changed, 5 insertions(+), 19 deletions(-) [+]
line wrap: on
line diff
--- a/mercurial/debugcommands.py	Mon Dec 14 16:15:01 2020 +0530
+++ b/mercurial/debugcommands.py	Wed Dec 16 12:39:15 2020 +0530
@@ -3933,7 +3933,7 @@
       * `--no-changelog --no-manifest --no-filelogs`: skip all filelog optimisation
     """
     return upgrade.upgraderepo(
-        ui, repo, run=run, optimize=optimize, backup=backup, **opts
+        ui, repo, run=run, optimize=set(optimize), backup=backup, **opts
     )
 
 
--- a/mercurial/upgrade.py	Mon Dec 14 16:15:01 2020 +0530
+++ b/mercurial/upgrade.py	Wed Dec 16 12:39:15 2020 +0530
@@ -22,19 +22,6 @@
 
 allformatvariant = upgrade_actions.allformatvariant
 
-# search without '-' to support older form on newer client.
-#
-# We don't enforce backward compatibility for debug command so this
-# might eventually be dropped. However, having to use two different
-# forms in script when comparing result is anoying enough to add
-# backward compatibility for a while.
-legacy_opts_map = {
-    b'redeltaparent': b're-delta-parent',
-    b'redeltamultibase': b're-delta-multibase',
-    b'redeltaall': b're-delta-all',
-    b'redeltafulladd': b're-delta-fulladd',
-}
-
 
 def upgraderepo(
     ui,
@@ -48,8 +35,7 @@
 ):
     """Upgrade a repository in place."""
     if optimize is None:
-        optimize = []
-    optimize = {legacy_opts_map.get(o, o) for o in optimize}
+        optimize = {}
     repo = repo.unfiltered()
 
     revlogs = set(upgrade_engine.UPGRADE_ALL_REVLOGS)
--- a/tests/test-upgrade-repo.t	Mon Dec 14 16:15:01 2020 +0530
+++ b/tests/test-upgrade-repo.t	Wed Dec 16 12:39:15 2020 +0530
@@ -211,7 +211,7 @@
 
 --optimize can be used to add optimizations
 
-  $ hg debugupgrade --optimize redeltaparent
+  $ hg debugupgrade --optimize 're-delta-parent'
   (no format upgrades found in existing repository)
   performing an upgrade with "--run" will make the following changes:
   
@@ -1060,7 +1060,7 @@
   removing temporary repository $TESTTMP/store-filenames/.hg/upgrade.* (glob)
   copy of old repository backed up at $TESTTMP/store-filenames/.hg/upgradebackup.* (glob)
   the old repository will not be deleted; remove it to free up disk space once the upgraded repository is verified
-  $ hg debugupgraderepo --run --optimize redeltafulladd
+  $ hg debugupgraderepo --run --optimize 're-delta-fulladd'
   upgrade will perform the following actions:
   
   requirements
@@ -1289,7 +1289,7 @@
         1       1        2        0      p1         21        191         98   0.51309        98         0    0.00000         98         98   1.00000        1
         2       1        2        0   other         30        200        107   0.53500       128        21    0.19626        128        128   0.83594        1
 
-  $ hg debugupgraderepo --run --optimize redeltaall
+  $ hg debugupgraderepo --run --optimize 're-delta-all'
   upgrade will perform the following actions:
   
   requirements