From 479ab249bb46a087dc4df844d7ee66a40aca9be2 Mon Sep 17 00:00:00 2001
From: Stein Magnus Jodal <stein.magnus@jodal.no>
Date: Tue, 16 Oct 2012 22:24:11 +0200
Subject: [PATCH] Move mopidy.utils.{log => formatting}.indent to break import
 cycle

---
 mopidy/frontends/mpd/session.py |  4 ++--
 mopidy/utils/deps.py            |  4 ++--
 mopidy/utils/formatting.py      |  8 ++++++++
 mopidy/utils/log.py             | 10 ----------
 mopidy/utils/settings.py        |  9 ++++-----
 5 files changed, 16 insertions(+), 19 deletions(-)
 create mode 100644 mopidy/utils/formatting.py

diff --git a/mopidy/frontends/mpd/session.py b/mopidy/frontends/mpd/session.py
index b4531c83..b5368a08 100644
--- a/mopidy/frontends/mpd/session.py
+++ b/mopidy/frontends/mpd/session.py
@@ -1,7 +1,7 @@
 import logging
 
 from mopidy.frontends.mpd import dispatcher, protocol
-from mopidy.utils import log, network
+from mopidy.utils import formatting, network
 
 logger = logging.getLogger('mopidy.frontends.mpd')
 
@@ -36,7 +36,7 @@ class MpdSession(network.LineProtocol):
         logger.debug(
             u'Response to [%s]:%s from %s: %s',
             self.host, self.port, self.actor_urn,
-            log.indent(self.terminator.join(response)))
+            formatting.indent(self.terminator.join(response)))
 
         self.send_lines(response)
 
diff --git a/mopidy/utils/deps.py b/mopidy/utils/deps.py
index d72f1392..32949f55 100644
--- a/mopidy/utils/deps.py
+++ b/mopidy/utils/deps.py
@@ -8,7 +8,7 @@ import gst
 
 import pykka
 
-from mopidy.utils.log import indent
+from . import formatting
 
 
 def list_deps_optparse_callback(*args):
@@ -47,7 +47,7 @@ def format_dependency_list(adapters=None):
                 os.path.dirname(dep_info['path'])))
         if 'other' in dep_info:
             lines.append('  Other: %s' % (
-                indent(dep_info['other'])),)
+                formatting.indent(dep_info['other'])),)
     return '\n'.join(lines)
 
 
diff --git a/mopidy/utils/formatting.py b/mopidy/utils/formatting.py
new file mode 100644
index 00000000..46459959
--- /dev/null
+++ b/mopidy/utils/formatting.py
@@ -0,0 +1,8 @@
+def indent(string, places=4, linebreak='\n'):
+    lines = string.split(linebreak)
+    if len(lines) == 1:
+        return string
+    result = u''
+    for line in lines:
+        result += linebreak + ' ' * places + line
+    return result
diff --git a/mopidy/utils/log.py b/mopidy/utils/log.py
index d5c9a14d..93f17c92 100644
--- a/mopidy/utils/log.py
+++ b/mopidy/utils/log.py
@@ -50,13 +50,3 @@ def setup_debug_logging_to_file():
     handler.setLevel(logging.DEBUG)
     root = logging.getLogger('')
     root.addHandler(handler)
-
-
-def indent(string, places=4, linebreak='\n'):
-    lines = string.split(linebreak)
-    if len(lines) == 1:
-        return string
-    result = u''
-    for line in lines:
-        result += linebreak + ' ' * places + line
-    return result
diff --git a/mopidy/utils/settings.py b/mopidy/utils/settings.py
index 39d613b3..6d868d39 100644
--- a/mopidy/utils/settings.py
+++ b/mopidy/utils/settings.py
@@ -9,8 +9,7 @@ import pprint
 import sys
 
 from mopidy import exceptions, SETTINGS_PATH, SETTINGS_FILE
-from mopidy.utils import log
-from mopidy.utils import path
+from mopidy.utils import formatting, path
 
 logger = logging.getLogger('mopidy.utils.settings')
 
@@ -76,7 +75,7 @@ class SettingsProxy(object):
         if self.get_errors():
             logger.error(
                 u'Settings validation errors: %s',
-                log.indent(self.get_errors_as_string()))
+                formatting.indent(self.get_errors_as_string()))
             raise exceptions.SettingsError(u'Settings validation failed.')
 
     def _read_missing_settings_from_stdin(self, current, runtime):
@@ -203,11 +202,11 @@ def format_settings_list(settings):
         default_value = settings.default.get(key)
         masked_value = mask_value_if_secret(key, value)
         lines.append(u'%s: %s' % (
-            key, log.indent(pprint.pformat(masked_value), places=2)))
+            key, formatting.indent(pprint.pformat(masked_value), places=2)))
         if value != default_value and default_value is not None:
             lines.append(
                 u'  Default: %s' %
-                log.indent(pprint.pformat(default_value), places=4))
+                formatting.indent(pprint.pformat(default_value), places=4))
         if errors.get(key) is not None:
             lines.append(u'  Error: %s' % errors[key])
     return '\n'.join(lines)