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

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):
@@ -23 +27 @@
 
     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)
 
@@ -35 +41 @@
 
     def local_url(self):
+        """Returns the local URL for this component"""
         return 'file://%s/%s' % (self.repo_dir, self.component)
 
@@ -48 +55 @@
 
     def version_subdir(self):
+        """Returns the subdirectory name for the version"""
         if self.version == 'trunk':
             subdir = 'trunk'
@@ -58 +66 @@
         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
+