extras-buildsys/server PackageJob.py,1.21.2.1,1.21.2.2

[Daniel Williams (dcbw)]

Author: dcbw

Update of /cvs/fedora/extras-buildsys/server
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv25134/server

Modified Files:
      Tag: STABLE_0_3
	PackageJob.py 
Log Message:
2005-08-23  Dan Williams <dcbw at redhat.com>

    * server/PackageJob.py
        - (_stage_checkout): fix typo in error message
        - (_cleanup_job_files): don't traceback cleaning up
            job files if we don't have any




Index: PackageJob.py
===================================================================
RCS file: /cvs/fedora/extras-buildsys/server/PackageJob.py,v
retrieving revision 1.21.2.1
retrieving revision 1.21.2.2
diff -u -r1.21.2.1 -r1.21.2.2
--- PackageJob.py	19 Aug 2005 19:34:57 -0000	1.21.2.1
+++ PackageJob.py	23 Aug 2005 20:16:57 -0000	1.21.2.2
@@ -295,7 +295,7 @@
         debugprint("%d: Running %s" % (self.uid, cmd))
         s, o = commands.getstatusoutput(cmd)
         if s != 0:
-            err_msg = "Erro: could not check out %s from %s - output was:\n\n%s" % (self.cvs_tag, self.target, o)
+            err_msg = "Error: could not check out %s from %s - output was:\n\n%s" % (self.cvs_tag, self.target, o)
         else:
             # Just in case the 'common' directory didn't come along for the ride,
             # get it from CVS
@@ -560,6 +560,9 @@
         self.bm.notify_job_done(self)
 
     def _cleanup_job_files(self):
+        if not self.result_dir or not self.srpm_http_path:
+            return
+
         srpm_file = os.path.join(self.result_dir, os.path.basename(self.srpm_http_path))
 
         # Delete any RPMs in the arch dirs