Changes in mergebot/trunk/mergebot/Actor.py [24:17]

File:

• mergebot/trunk/mergebot/Actor.py (modified) (5 diffs)

mergebot/trunk/mergebot/Actor.py

@@ -1 @@
-"""Base class for mergebot actors that do the various kinds of tasks
-"""
 import os
-from mergebot import SvnOps
 
 class Actor(object):
-    """Base class for mergebot actors"""
     def __init__(self, work_dir, repo_url, repo_dir, ticket, component,
                  version, summary, requestor):
@@ -27 +23 @@
 
     def logfilename(self):
-        """Returns the absolute path of the logfile for this ticket"""
         return os.path.abspath(os.path.join(os.path.dirname(self.work_dir),
             'ticket-%s.log' % self.ticket))
 
     def public_url(self):
-        """Returns the public URL for this component"""
         return '%s/%s' % (self.repo_url, self.component)
 
@@ -41 +35 @@
 
     def local_url(self):
-        """Returns the local URL for this component"""
         return 'file://%s/%s' % (self.repo_dir, self.component)
 
@@ -55 +48 @@
 
     def version_subdir(self):
-        """Returns the subdirectory name for the version"""
         if self.version == 'trunk':
             subdir = 'trunk'
@@ -66 +58 @@
         return subdir
 
-    def check_required_directories(self):
-        """Make sure the various urls we require do exist"""
-        if not SvnOps.does_url_exist(self.local_url()):
-            return 'Component %s does not exist in the repository.' \
-                % self.component
-        if not SvnOps.does_url_exist(self.local_url() + '/branches'):
-            return 'No directory in which to create branches for ' \
-                'component %s in the repository.' % self.component
-        if not SvnOps.does_url_exist(self.baseline_local_url()):
-            return 'Version %s for component %s does not exist in the ' \
-                'repository.' % (self.version, self.component)
-        return None
-