extras-repoclosure PackageOwners.py,1.5,1.6 rc-report.py,1.19,1.20

[Michael Schwendt (mschwendt)]

Author: mschwendt

Update of /cvs/fedora/extras-repoclosure
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv13840/extras-repoclosure

Modified Files:
	PackageOwners.py rc-report.py 
Log Message:
get rid of the sys.exit() calls in PackageOwners.py



Index: PackageOwners.py
===================================================================
RCS file: /cvs/fedora/extras-repoclosure/PackageOwners.py,v
retrieving revision 1.5
retrieving revision 1.6
diff -u -r1.5 -r1.6
--- PackageOwners.py	22 Sep 2006 12:26:38 -0000	1.5
+++ PackageOwners.py	22 Sep 2006 13:18:29 -0000	1.6
@@ -145,7 +145,7 @@
             if rc:
                 # TODO: customise behaviour on error conditions
                 self._leaveworkdir()
-                sys.exit(1)
+                return False
 
         try:
             f = file( self.ownersfile )
@@ -153,7 +153,7 @@
             print 'ERROR: %s' % strerr
             # TODO: customise behaviour on error conditions
             self._leaveworkdir()
-            sys.exit(err)
+            return err
         ownerslist = f.readlines()
         f.close()
         self._parse(ownerslist)
@@ -173,7 +173,7 @@
         if rc:
             # TODO: customise behaviour on error conditions
             print 'ERROR: Could not download owners.list from %s' % self.url
-            sys.exit(1)
+            return 1
         else:
             ownerslist = f.readlines()
             f.close()


Index: rc-report.py
===================================================================
RCS file: /cvs/fedora/extras-repoclosure/rc-report.py,v
retrieving revision 1.19
retrieving revision 1.20
diff -u -r1.19 -r1.20
--- rc-report.py	22 Sep 2006 12:26:38 -0000	1.19
+++ rc-report.py	22 Sep 2006 13:18:29 -0000	1.20
@@ -179,7 +179,8 @@
 
 owners = PackageOwners()
 #owners.FromCVS(workdir=ownersworkdir)
-owners.FromURL()
+if not owners.FromURL():
+    sys.exit(1)
 summail = ''
 pkgbyowner = {}  # map, pkgbyowner[email], list of pkgids for all owner's broken packages
 summary = {}  # map of maps, summary[repoid][pkgid], unresolved deps line by line