convert/svn: drop undocumented and untested revision 'blacklist'
authorPatrick Mezard <pmezard@gmail.com>
Sun, 21 Jun 2009 23:17:05 +0200
changeset 8888 bd93d0e0d317
parent 8887 0332f8b44e54
child 8889 9be824115ee8
convert/svn: drop undocumented and untested revision 'blacklist'
hgext/convert/subversion.py
--- a/hgext/convert/subversion.py	Sun Jun 21 23:17:05 2009 +0200
+++ b/hgext/convert/subversion.py	Sun Jun 21 23:17:05 2009 +0200
@@ -249,11 +249,6 @@
             raise util.Abort(_('svn: start revision %s is not an integer')
                              % self.startrev)
 
-        try:
-            self.get_blacklist()
-        except IOError:
-            pass
-
         self.head = self.latest(self.module, latest)
         if not self.head:
             raise util.Abort(_('no revision found in module %s')
@@ -576,26 +571,6 @@
             return None
         return self.revid(dirent.created_rev, path)
 
-    def get_blacklist(self):
-        """Avoid certain revision numbers.
-        It is not uncommon for two nearby revisions to cancel each other
-        out, e.g. 'I copied trunk into a subdirectory of itself instead
-        of making a branch'. The converted repository is significantly
-        smaller if we ignore such revisions.
-        """
-        self.blacklist = set()
-        blacklist = self.blacklist
-        for line in file("blacklist.txt", "r"):
-            if not line.startswith("#"):
-                try:
-                    svn_rev = int(line.strip())
-                    blacklist.add(svn_rev)
-                except ValueError:
-                    pass # not an integer or a comment
-
-    def is_blacklisted(self, svn_rev):
-        return svn_rev in self.blacklist
-
     def reparent(self, module):
         """Reparent the svn transport and return the previous parent."""
         if self.prevmodule == module:
@@ -809,10 +784,6 @@
                     if revnum < self.startrev:
                         lastonbranch = True
                         break
-                    if self.is_blacklisted(revnum):
-                        self.ui.note(_('skipping blacklisted revision %d\n')
-                                     % revnum)
-                        continue
                     if not paths:
                         self.ui.debug(_('revision %d has no entries\n') % revnum)
                         continue