From 5b33e4047e9c78147aab2cca07b1dee5f122c5ce Mon Sep 17 00:00:00 2001
From: Romain Bignon <romain@peerfuse.org>
Date: Mon, 15 Nov 2010 23:21:28 +0100
Subject: [PATCH] print call errors to stderr

---
 scripts/boobank-munin            | 12 ++++++++++--
 weboob/tools/application/base.py |  7 -------
 2 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/scripts/boobank-munin b/scripts/boobank-munin
index 446abb41..936e9391 100755
--- a/scripts/boobank-munin
+++ b/scripts/boobank-munin
@@ -20,6 +20,7 @@
 import os
 import sys
 import time
+import logging
 from weboob.core import Weboob, CallErrors
 from weboob.capabilities.bank import ICapBank
 
@@ -160,7 +161,8 @@ class BoobankMuninPlugin(object):
                 self.write_output('%s.label %s' % (id, account.label.encode('iso-8859-15')))
                 if self.cumulate:
                     self.write_output('%s.draw %s' % (id, type))
-        except CallErrors:
+        except CallErrors, errors:
+            self.print_errors(errors)
             self.print_cache('boobank-munin-config')
         else:
             self.flush_cache()
@@ -171,6 +173,10 @@ class BoobankMuninPlugin(object):
     def account2id(self, account):
         return '%s_%s' % (account.backend, account.id)
 
+    def print_errors(self, errors):
+        for backend, err, backtrace in errors:
+            print >>sys.stderr, '%s(%s): %s' % (type(err).__name__, backend.name, err)
+
     def execute(self):
         if self.check_cache('boobank-munin'):
             return
@@ -184,7 +190,8 @@ class BoobankMuninPlugin(object):
                     if account.coming and self.add_coming:
                         balance += account.coming
                     self.write_output('%s.value %d' % (self.account2id(account), balance))
-        except CallErrors:
+        except CallErrors, errors:
+            self.print_errors(errors)
             self.print_cache('boobank-munin')
         else:
             self.flush_cache()
@@ -212,4 +219,5 @@ class BoobankMuninPlugin(object):
         sys.exit(0)
 
 if __name__ == '__main__':
+    logging.basicConfig()
     BoobankMuninPlugin().run()
diff --git a/weboob/tools/application/base.py b/weboob/tools/application/base.py
index edc1b952..d86da018 100644
--- a/weboob/tools/application/base.py
+++ b/weboob/tools/application/base.py
@@ -325,13 +325,6 @@ class BaseApplication(object):
             handler.setLevel(level)
             logging.root.addHandler(handler)
 
-        #log_format = '%(asctime)s:%(levelname)s:%(name)s:%(filename)s:%(lineno)d:%(funcName)s %(message)s'
-        #if self.options.logging_file:
-        #    print self.options.logging_file
-        #    logging.basicConfig(filename=self.options.logging_file, level=level, format=log_format)
-        #else:
-        #    logging.basicConfig(stream=sys.stdout, level=level, format=log_format)
-
         self._handle_options()
         self.handle_application_options()