do not use pprint, avoid filter()

stroeder · stroeder · commit 37983e2fbbe7 · 2017-11-20T15:54:01.000Z
diff --git a/setup.py b/setup.py
@@ -6,7 +6,6 @@
 
 import sys
 import os
-import pprint
 from ConfigParser import ConfigParser
 
 # Python 2.3.6+ and setuptools are needed to build eggs, so
@@ -49,14 +48,13 @@ def __init__(self, meta_defines):
         cfg.read('setup.cfg')
         _ldap_cfg = dict(cfg.items('_ldap'))
         for name, value in _ldap_cfg.items():
-            _ldap_cfg[name] = filter(None, value.split(' '))
+            _ldap_cfg[name] = [ val for val in value.split(' ') if val ]
         # split values of extra_files
         if 'extra_files' in _ldap_cfg:
             for i in range(len(_ldap_cfg['extra_files'])):
                 destdir, origfiles = self.extra_files[i].split(':')
                 origfileslist = origfiles.split(',')
                 _ldap_cfg['extra_files'][i] = (destdir, origfileslist)
-        #pprint.pprint(_ldap_cfg)
         for name, val in _ldap_cfg.items():
             setattr(self, name, val)
         if 'ldap_r' in self.libs or 'oldap_r' in self.libs:
@@ -83,8 +81,6 @@ def __init__(self, meta_defines):
     ],
 )
 
-pprint.pprint(LDAP_CLASS.__dict__)
-
 
 #-- Let distutils/setuptools do the rest
 
