extras-repoclosure PackageOwners.py,1.6,1.7

[Michael Schwendt (mschwendt)]

Author: mschwendt

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

Modified Files:
	PackageOwners.py 
Log Message:



Index: PackageOwners.py
===================================================================
RCS file: /cvs/fedora/extras-repoclosure/PackageOwners.py,v
retrieving revision 1.6
retrieving revision 1.7
diff -u -r1.6 -r1.7
--- PackageOwners.py	22 Sep 2006 13:18:29 -0000	1.6
+++ PackageOwners.py	22 Sep 2006 13:22:34 -0000	1.7
@@ -35,7 +35,7 @@
         self.url = url
         self.retries = retries
         self.retrysecs = retrysecs
-        self._refresh()
+        return self._refresh()
     
     
     def FromCVS(self, retries=3, retrysecs=300, command='LC_ALL=C CVS_RSH=ssh cvs -f -d :pserver:anonymous at cvs.fedora.redhat.com:/cvs/extras co owners', workdir=''):
@@ -46,7 +46,7 @@
         self.workdir = workdir
         self.ownersfile = os.path.join('owners', 'owners.list')
         self.cwdstack = []
-        self._refresh()
+        return self._refresh()
 
 
     def __getitem__(self,rpmname):
@@ -94,7 +94,7 @@
     
     def _refresh(self):
         self.dict = {}  # map package name to email address, dict[name]
-        self._download()
+        return self._download()
 
 
     def _parse(self,ownerslist):
@@ -158,6 +158,7 @@
         f.close()
         self._parse(ownerslist)
         self._leaveworkdir()
+        return True
 
 
     def _downloadfromurl(self):
@@ -173,19 +174,21 @@
         if rc:
             # TODO: customise behaviour on error conditions
             print 'ERROR: Could not download owners.list from %s' % self.url
-            return 1
+            return False
         else:
             ownerslist = f.readlines()
             f.close()
             self._parse(ownerslist)
+            return True
 
 
     def _download(self):
         if self.how == 'url':
-            self._downloadfromurl()
+            return self._downloadfromurl()
         elif self.how == 'cvs':
-            self._downloadfromcvs()
+            return self._downloadfromcvs()
         else:
             self.__init__()
+            return False