From b056804f3d3e3a2d92dac42062527873255d14c7 Mon Sep 17 00:00:00 2001
From: Sol Jerome <sol.jerome@gmail.com>
Date: Sun, 17 Oct 2010 20:43:16 -0500
Subject: Client Tools: Pylint/PEP8 fixes

Signed-off-by: Sol Jerome <sol.jerome@gmail.com>
---
 src/lib/Client/Tools/POSIX.py | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

(limited to 'src/lib/Client/Tools/POSIX.py')

diff --git a/src/lib/Client/Tools/POSIX.py b/src/lib/Client/Tools/POSIX.py
index 9d37b77d3..a4417297a 100644
--- a/src/lib/Client/Tools/POSIX.py
+++ b/src/lib/Client/Tools/POSIX.py
@@ -44,6 +44,7 @@ def calcPerms(initial, perms):
                 tempperms |= perm
     return tempperms
 
+
 def normUid(entry):
     """
        This takes a user name or uid and
@@ -58,6 +59,7 @@ def normUid(entry):
         log.error('UID normalization failed for %s' % (entry.get('name')))
         return False
 
+
 def normGid(entry):
     """
        This takes a group name or gid and
@@ -75,6 +77,7 @@ def normGid(entry):
 text_chars = "".join([chr(y) for y in range(32, 127)] + list("\n\r\t\b"))
 notrans = string.maketrans("", "")
 
+
 def isString(strng):
     """Returns true if a string contains no binary chars."""
     if "\0" in strng:
@@ -85,6 +88,7 @@ def isString(strng):
 
     return len(strng.translate(notrans, text_chars)) == 0
 
+
 class POSIX(Bcfg2.Client.Tools.Tool):
     """POSIX File support code."""
     name = 'POSIX'
@@ -161,7 +165,8 @@ class POSIX(Bcfg2.Client.Tools.Tool):
                 else:
                     os.unlink(entry.get('name'))
             except OSError:
-                self.logger.info("Symlink %s cleanup failed" % (entry.get('name')))
+                self.logger.info("Symlink %s cleanup failed" %\
+                                 (entry.get('name')))
         try:
             os.symlink(entry.get('to'), entry.get('name'))
             return True
@@ -205,7 +210,8 @@ class POSIX(Bcfg2.Client.Tools.Tool):
         pruneTrue = True
         ex_ents = []
         if entry.get('prune', 'false') == 'true' \
-               and (entry.tag == 'Directory' or entry.get('type') == 'directory'):
+               and (entry.tag == 'Directory' or
+                    entry.get('type') == 'directory'):
                # FIXME: need to verify both old and new POSIX types
             try:
                 entries = ['/'.join([entry.get('name'), ent]) \
@@ -217,7 +223,8 @@ class POSIX(Bcfg2.Client.Tools.Tool):
                                       entry.get('name'))
                     self.logger.debug(ex_ents)
                     nqtext = entry.get('qtext', '') + '\n'
-                    nqtext += "Directory %s contains extra entries:" % entry.get('name')
+                    nqtext += "Directory %s contains extra entries:" % \
+                              entry.get('name')
                     nqtext += ":".join(ex_ents)
                     entry.set('qtest', nqtext)
                     [entry.append(XML.Element('Prune', path=x)) for x in ex_ents]
-- 
cgit v1.2.3-1-g7c22