diff mercurial/upgrade_utils/engine.py @ 46575:636853347e14

upgrade: write nodemap for manifests too In 98e39f04d60e I assumed that writing nodemap for manifests was not desirable and stopped writing it during upgrade. However in recent discussion with Pierre-Yves, I learnt that that's not true. Differential Revision: https://phab.mercurial-scm.org/D9991
author Pulkit Goyal <7895pulkit@gmail.com>
date Mon, 15 Feb 2021 15:13:20 +0530
parents 98e39f04d60e
children 67b5fafd3a46
line wrap: on
line diff
--- a/mercurial/upgrade_utils/engine.py	Tue Feb 23 12:29:41 2021 -0800
+++ b/mercurial/upgrade_utils/engine.py	Mon Feb 15 15:13:20 2021 +0530
@@ -468,6 +468,13 @@
             unfi = srcrepo.unfiltered()
             cl = unfi.changelog
             nodemap.persist_nodemap(tr, cl, force=True)
+            # we want to directly operate on the underlying revlog to force
+            # create a nodemap file. This is fine since this is upgrade code
+            # and it heavily relies on repository being revlog based
+            # hence accessing private attributes can be justified
+            nodemap.persist_nodemap(
+                tr, unfi.manifestlog._rootstore._revlog, force=True
+            )
         scmutil.writereporequirements(srcrepo, upgrade_op.new_requirements)
     else:
         with dstrepo.transaction(b'upgrade') as tr: