changeset 6795:1b59ddda3242 stable

topic: further "help pytype" by not mentioning remotebranchcache on hg 4.9 Because it doesn't exist. This obviously was a real issue on Mercurial 4.9 because we were trying to directly reference a class that doesn't exist in branchmap module, it wasn't a string in some typing comment or whatever, so this code previously would not work on 4.9 at all. Oops!
author Anton Shestakov <av6@dwimlabs.net>
date Thu, 27 Jun 2024 17:14:50 +0400
parents 68f7ba35ea83
children 84149bb93f9d
files hgext3rd/topic/topicmap.py
diffstat 1 files changed, 9 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/hgext3rd/topic/topicmap.py	Thu Jun 27 16:59:29 2024 +0400
+++ b/hgext3rd/topic/topicmap.py	Thu Jun 27 17:14:50 2024 +0400
@@ -180,6 +180,12 @@
 else:
     oldbranchmap = util.nullcontextmanager
 
+if util.safehasattr(branchmap, 'branchcache'):
+    allbccls = (branchmap.branchcache,)
+    if util.safehasattr(branchmap, 'remotebranchcache'):
+        # hg <= 4.9 (eb7ce452e0fb)
+        allbccls = (branchmap.branchcache, branchmap.remotebranchcache)
+
 class _topiccache(object): # combine me with branchmap.branchcache
 
     def __init__(self, *args, **kwargs):
@@ -189,7 +195,7 @@
 
     def copy(self):
         """return an deep copy of the branchcache object"""
-        assert isinstance(self, (branchmap.branchcache, branchmap.remotebranchcache))  # help pytype
+        assert isinstance(self, allbccls)  # help pytype
         entries = compat.bcentries(self)
         args = (entries, self.tipnode, self.tiprev, self.filteredhash,
                 self._closednodes)
@@ -218,7 +224,7 @@
 
         - False when cached tipnode is unknown or if we detect a strip.
         - True when cache is up to date or a subset of current repo."""
-        assert isinstance(self, (branchmap.branchcache, branchmap.remotebranchcache))  # help pytype
+        assert isinstance(self, allbccls)  # help pytype
         valid = super(_topiccache, self).validfor(repo)
         if not valid:
             return False
@@ -255,7 +261,7 @@
         missing heads, and a generator of nodes that are strictly a superset of
         heads missing, this function updates self to be correct.
         """
-        assert isinstance(self, (branchmap.branchcache, branchmap.remotebranchcache))  # help pytype
+        assert isinstance(self, allbccls)  # help pytype
         if not istopicfilter(repo.filtername):
             return super(_topiccache, self).update(repo, revgen)