Use the standard logging module

Get rid of the gbp-specific logging in order to diminish dependency to
gbp specifics.

Change-Id: I39cfb5c3727b5828e63c7932f1dbc0b926d5bf99
Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

diff --git a/obs_service_gbp/__init__.py b/obs_service_gbp/__init__.py
index bfb57d9514a981e0c402afb16c075fbe92cc95a2..43cb263440480ef1bbe253d11af8db40f806e31b 100644 (file)
--- a/obs_service_gbp/__init__.py
+++ b/obs_service_gbp/__init__.py
@@ -17,11 +17,3 @@
 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
 # MA 02110-1301, USA.
 """Helpers for the git-buildpackage OBS source service"""
-
-import gbp.log as gbplog
-
-
-# Setup logging
-LOGGER = gbplog.getLogger('source_service')
-LOGGER.setLevel(gbplog.INFO)
-

diff --git a/obs_service_gbp/command.py b/obs_service_gbp/command.py
index d5ab1fe78323cfe8c673697d6ddc8cc1463dbf6f..768c23504375a9253264b010c1f4013baab5749a 100644 (file)
--- a/obs_service_gbp/command.py
+++ b/obs_service_gbp/command.py
@@ -20,20 +20,26 @@
 
 import os
 import argparse
+import logging
 import shutil
 import tempfile
 from ConfigParser import SafeConfigParser
 
+import gbp.log as gbplog
 from gbp.rpm import guess_spec, NoSpecError
 from gbp.scripts.buildpackage import main as gbp_deb
 from gbp.scripts.buildpackage_rpm import main as gbp_rpm
 
-from obs_service_gbp import LOGGER, gbplog
 from obs_service_gbp_utils import (GbpServiceError, GbpChildBTError, fork_call,
             sanitize_uid_gid, write_treeish_meta, str_to_bool)
 from gbp_repocache import CachedRepo, CachedRepoError
 import gbp_repocache
 
+
+# Setup module-level logging
+LOGGER = logging.getLogger('source_service')
+
+
 def have_spec(directory):
     """Find if the package has spec files"""
     try:
@@ -192,11 +198,13 @@ def main(argv=None):
 
     args = parse_args(argv)
 
+    logging.basicConfig(level=logging.INFO,
+                        format='%(name)s:%(levelname)s: %(message)s')
     LOGGER.info('Starting git-buildpackage source service')
     if args.verbose == 'yes':
         gbplog.setup(color='auto', verbose=True)
-        LOGGER.setLevel(gbplog.DEBUG)
-        gbp_repocache.LOGGER.setLevel(gbplog.DEBUG)
+        logging.root.setLevel(logging.DEBUG)
+        gbp_repocache.LOGGER.setLevel(logging.DEBUG)
 
     config = read_config(args.config)