manifest: define and implement rev() on manifestlog
authorGregory Szorc <gregory.szorc@gmail.com>
Wed, 04 Jul 2018 12:12:49 -0700
changeset 38556 f2f9bacf0587
parent 38555 f83600efa1ca
child 38557 0db41eb0a3ac
manifest: define and implement rev() on manifestlog Various code is accessing repo.manifestlog._revlog - a private attribute. This bypasses our interface and makes it difficult to implement non-revlog manifest storage. This commit adds a rev() method to the manifestlog interface and class and teaches callers to use it. Ideally this method wouldn't exist, as very few consumers should need to resolve the revision number of a manifest. Again, the primary goal of interface work is to establish and use interfaces first and to improve them later. Differential Revision: https://phab.mercurial-scm.org/D3882
mercurial/logcmdutil.py
mercurial/manifest.py
mercurial/repository.py
mercurial/templatekw.py
--- a/mercurial/logcmdutil.py	Mon Jul 02 11:14:13 2018 -0700
+++ b/mercurial/logcmdutil.py	Wed Jul 04 12:12:49 2018 -0700
@@ -228,7 +228,7 @@
 
         if self.ui.debugflag and rev is not None:
             mnode = ctx.manifestnode()
-            mrev = self.repo.manifestlog._revlog.rev(mnode)
+            mrev = self.repo.manifestlog.rev(mnode)
             self.ui.write(columns['manifest']
                           % scmutil.formatrevnode(self.ui, mrev, mnode),
                           label='ui.debug log.manifest')
--- a/mercurial/manifest.py	Mon Jul 02 11:14:13 2018 -0700
+++ b/mercurial/manifest.py	Wed Jul 04 12:12:49 2018 -0700
@@ -1348,6 +1348,9 @@
         self._dirmancache.clear()
         self._revlog.clearcaches()
 
+    def rev(self, node):
+        return self._revlog.rev(node)
+
 @interfaceutil.implementer(repository.imanifestrevisionwritable)
 class memmanifestctx(object):
     def __init__(self, manifestlog):
--- a/mercurial/repository.py	Mon Jul 02 11:14:13 2018 -0700
+++ b/mercurial/repository.py	Wed Jul 04 12:12:49 2018 -0700
@@ -922,6 +922,12 @@
     def clearcaches():
         """Clear caches associated with this collection."""
 
+    def rev(node):
+        """Obtain the revision number for a binary node.
+
+        Raises ``error.LookupError`` if the node is not known.
+        """
+
 class completelocalrepository(interfaceutil.Interface):
     """Monolithic interface for local repositories.
 
--- a/mercurial/templatekw.py	Mon Jul 02 11:14:13 2018 -0700
+++ b/mercurial/templatekw.py	Wed Jul 04 12:12:49 2018 -0700
@@ -465,7 +465,7 @@
     if mnode is None:
         # just avoid crash, we might want to use the 'ff...' hash in future
         return
-    mrev = repo.manifestlog._revlog.rev(mnode)
+    mrev = repo.manifestlog.rev(mnode)
     mhex = hex(mnode)
     mapping = context.overlaymap(mapping, {'rev': mrev, 'node': mhex})
     f = context.process('manifest', mapping)