phases: updatephases should not skip internal phase
authorJoerg Sonnenberger <joerg@bec.de>
Tue, 07 Jul 2020 22:00:02 +0200
changeset 45116 361a7444bc41
parent 45115 672ad1f6eeb8
child 45117 b1e51ef4e536
phases: updatephases should not skip internal phase Historically, updatephases used to skip the secret phase, but wasn't updated when archived and internal were added. Differential Revision: https://phab.mercurial-scm.org/D8696
mercurial/phases.py
--- a/mercurial/phases.py	Sun Jun 28 01:08:13 2020 +0200
+++ b/mercurial/phases.py	Tue Jul 07 22:00:02 2020 +0200
@@ -777,12 +777,12 @@
 
 def updatephases(repo, trgetter, headsbyphase):
     """Updates the repo with the given phase heads"""
-    # Now advance phase boundaries of all but secret phase
+    # Now advance phase boundaries of all phases
     #
     # run the update (and fetch transaction) only if there are actually things
     # to update. This avoid creating empty transaction during no-op operation.
 
-    for phase in allphases[:-1]:
+    for phase in allphases:
         revset = b'%ln - _phase(%s)'
         heads = [c.node() for c in repo.set(revset, headsbyphase[phase], phase)]
         if heads: