changeset 35342:75520786ad2f

upgrade: use the repository 'ui' as the base for the new repository The `repo.baseui` contains all the configuration but the one specific to the repository (so it can be used when dealing with local peer and sub- repository). However, we need the repository config to be taken into account when doing the upgrade. Otherwise, the upgrade related config that exists in the repository config won't be taken into account when performing the update. A buggy and surprising behavior. We had to work around protection set around `repo.ui.copy` since we are an uncommon case.
author Boris Feld <boris.feld@octobus.net>
date Thu, 07 Dec 2017 18:55:35 +0100
parents cb7a0ae053c6
children 06987c6971be
files mercurial/upgrade.py tests/test-upgrade-repo.t
diffstat 2 files changed, 19 insertions(+), 6 deletions(-) [+]
line wrap: on
line diff
--- a/mercurial/upgrade.py	Thu Dec 07 20:50:24 2017 +0100
+++ b/mercurial/upgrade.py	Thu Dec 07 18:55:35 2017 +0100
@@ -830,9 +830,22 @@
         try:
             ui.write(_('creating temporary repository to stage migrated '
                        'data: %s\n') % tmppath)
-            dstrepo = localrepo.localrepository(repo.baseui,
-                                                path=tmppath,
-                                                create=True)
+
+            # repo.ui is protected against copy:
+            #
+            # running `repo.ui.copy` actually call `repo.baseui.copy`. Here, we
+            # -really- wants to copy the actual `repo.ui` object (since we
+            # create a copy of the repository).
+            #
+            # We have to work around the protection.
+            oldcopy = repo.ui.copy
+            try:
+                repo.ui.__dict__.pop('copy', None)
+                dstrepo = localrepo.localrepository(repo.ui,
+                                                    path=tmppath,
+                                                    create=True)
+            finally:
+                repo.ui.copy = oldcopy
 
             with dstrepo.wlock(), dstrepo.lock():
                 backuppath = _upgraderepo(ui, repo, dstrepo, newreqs,
--- a/tests/test-upgrade-repo.t	Thu Dec 07 20:50:24 2017 +0100
+++ b/tests/test-upgrade-repo.t	Thu Dec 07 18:55:35 2017 +0100
@@ -531,12 +531,12 @@
   migrating 9 total revisions (3 in filelogs, 3 in manifests, 3 in changelog)
   migrating 497 bytes in store; 882 bytes tracked data
   migrating 1 filelogs containing 3 revisions (182 bytes in store; 573 bytes tracked data)
-  finished migrating 3 filelog revisions across 1 filelogs; change in size: 0 bytes
+  finished migrating 3 filelog revisions across 1 filelogs; change in size: -63 bytes
   migrating 1 manifests containing 3 revisions (141 bytes in store; 138 bytes tracked data)
   finished migrating 3 manifest revisions across 1 manifests; change in size: 0 bytes
   migrating changelog containing 3 revisions (174 bytes in store; 171 bytes tracked data)
   finished migrating 3 changelog revisions; change in size: 0 bytes
-  finished migrating 9 total revisions; total change in store size: 0 bytes
+  finished migrating 9 total revisions; total change in store size: -63 bytes
   copying phaseroots
   data fully migrated to temporary repository
   marking source repository as being upgraded; clients will be unable to read from repository
@@ -552,7 +552,7 @@
      rev    offset  length  delta linkrev nodeid       p1           p2
        0         0      77     -1       0 bcc1d3df78b2 000000000000 000000000000
        1        77      21      0       1 af3e29f7a72e bcc1d3df78b2 000000000000
-       2        98      84     -1       2 8daf79c5522b af3e29f7a72e 000000000000
+       2        98      21      1       2 8daf79c5522b af3e29f7a72e 000000000000
   $ cd ..
 
   $ cat << EOF >> $HGRCPATH