Blob Blame History Raw
commit c8c4065931bec55e9b2eb0f16a97376e8650846b
Author: Radek Vykydal <rvykydal@redhat.com>
Date:   Wed Aug 10 11:10:58 2016 +0200

    Report __del__ RepoError exceptions into log instead of stderr (#1356797)
    
    Resolves: rhbz#1356797
    
    So it does not clutter text UI of clients like Anaconda.

diff --git a/yum/__init__.py b/yum/__init__.py
index 57e1dfe..9e38320 100644
--- a/yum/__init__.py
+++ b/yum/__init__.py
@@ -234,11 +234,14 @@ class YumBase(depsolve.Depsolve):
         self.updateinfo_filters = {}
 
     def __del__(self):
-        self.close()
-        self.closeRpmDB()
-        self.doUnlock()
-        # call cleanup callbacks
-        for cb in self._cleanup: cb()
+        try:
+            self.close()
+            self.closeRpmDB()
+            self.doUnlock()
+            # call cleanup callbacks
+            for cb in self._cleanup: cb()
+        except Errors.RepoError, e:
+            self.verbose_logger.debug("Exception %s %s in %s ignored" % (repr(e), str(e), self.__del__))
 
     def close(self):
         """Close the history and repo objects."""
diff --git a/yum/repos.py b/yum/repos.py
index a0ef28c..017527a 100644
--- a/yum/repos.py
+++ b/yum/repos.py
@@ -161,7 +161,10 @@ class RepoStorage:
         return str(self.repos.keys())
 
     def __del__(self):
-        self.close()
+        try:
+            self.close()
+        except Errors.RepoError, e:
+            self.logger.debug("Exception %s %s in %s ignored" % (repr(e), str(e), self.__del__))
 
     def close(self):
         for repo in self.repos.values():
@@ -423,7 +426,10 @@ class Repository:
         return hash(self.id)
         
     def __del__(self):
-        self.close()
+        try:
+            self.close()
+        except Errors.RepoError, e:
+            self.logger.debug("Exception %s %s in %s ignored" % (repr(e), str(e), self.__del__))
 
     def _ui_id(self):
         """ Show self.id, so we can use it and override it. """
diff --git a/yum/yumRepo.py b/yum/yumRepo.py
index 9c3d274..2db8faf 100644
--- a/yum/yumRepo.py
+++ b/yum/yumRepo.py
@@ -114,7 +114,10 @@ class YumPackageSack(packageSack.PackageSack):
         self.added = {}
 
     def __del__(self):
-        self.close()
+        try:
+            self.close()
+        except Errors.RepoError, e:
+            verbose_logger.debug("Exception %s %s in %s ignored" % (repr(e), str(e), self.__del__))
 
     def close(self):
         self.added = {}