capture ioerror/oserrors on handling metadata files and emit a proper MDError

fixes rh bug: https://bugzilla.redhat.com/show_bug.cgi?id=514995

diff --git a/createrepo/__init__.py b/createrepo/__init__.py
index 609569a81abac203bb3e2b455374dfac2b48cd6d..6df90991a57eee1fdc82af38d9af4c8de93c0191 100644 (file)
--- a/createrepo/__init__.py
+++ b/createrepo/__init__.py
@@ -362,9 +362,12 @@ class MetaDataGenerator:
         if not isinstance(packages, MetaSack):
             packages = self.trimRpms(packages)
         self.pkgcount = len(packages)
-        self.openMetadataDocs()
-        self.writeMetadataDocs(packages)
-        self.closeMetadataDocs()
+        try:
+            self.openMetadataDocs()
+            self.writeMetadataDocs(packages)
+            self.closeMetadataDocs()
+        except (IOError, OSError), e:
+            raise MDError, _('Cannot access/write repodata files: %s') % e
 
     def openMetadataDocs(self):
         if self.conf.database_only:
@@ -1133,14 +1136,17 @@ class SplitMetaDataGenerator(MetaDataGenerator):
         mediano = 1
         self.current_pkg = 0
         self.conf.baseurl = self._getFragmentUrl(self.conf.baseurl, mediano)
-        self.openMetadataDocs()
-        original_basedir = self.conf.basedir
-        for mydir in self.conf.directories:
-            self.conf.baseurl = self._getFragmentUrl(self.conf.baseurl, mediano)
-            self.writeMetadataDocs(filematrix[mydir], mydir)
-            mediano += 1
-        self.conf.baseurl = self._getFragmentUrl(self.conf.baseurl, 1)
-        self.closeMetadataDocs()
+        try:
+            self.openMetadataDocs()
+            original_basedir = self.conf.basedir
+            for mydir in self.conf.directories:
+                self.conf.baseurl = self._getFragmentUrl(self.conf.baseurl, mediano)
+                self.writeMetadataDocs(filematrix[mydir], mydir)
+                mediano += 1
+            self.conf.baseurl = self._getFragmentUrl(self.conf.baseurl, 1)
+            self.closeMetadataDocs()
+        except (IOError, OSError), e:
+            raise MDError, _('Cannot access/write repodata files: %s') % e