Stricter PEP-8 conformance. master
authorJoe Wreschnig <joe.wreschnig@gmail.com>
Mon, 24 Aug 2015 20:07:17 +0000 (22:07 +0200)
committerJoe Wreschnig <joe.wreschnig@gmail.com>
Mon, 24 Aug 2015 20:07:17 +0000 (22:07 +0200)
setup.py

index a6c07cd..f49d723 100755 (executable)
--- a/setup.py
+++ b/setup.py
@@ -9,31 +9,38 @@ from distutils.core import setup, Command, Extension
 from distutils.command.clean import clean as distutils_clean
 from distutils.command.sdist import sdist as distutils_sdist
 
+
 class clean(distutils_clean):
     def run(self):
         # In addition to what the normal clean run does, remove pyc
         # and pyo and backup files from the source tree.
         distutils_clean.run(self)
+
         def should_remove(filename):
-            if (filename.lower()[-4:] in [".pyc", ".pyo"] or
-                filename.endswith("~") or
-                (filename.startswith("#") and filename.endswith("#"))):
+            if (filename.lower()[-4:] in [".pyc", ".pyo"]
+                or filename.endswith("~")
+                or (filename.startswith("#")
+                    and filename.endswith("#"))):
                 return True
             else:
                 return False
         for pathname, dirs, files in os.walk(os.path.dirname(__file__)):
             for filename in filter(should_remove, files):
-                try: os.unlink(os.path.join(pathname, filename))
+                try:
+                    os.unlink(os.path.join(pathname, filename))
                 except EnvironmentError as err:
                     print(str(err))
 
-        try: os.unlink("MANIFEST")
-        except OSError: pass
+        try:
+            os.unlink("MANIFEST")
+        except OSError:
+            pass
 
         for base in ["coverage", "build", "dist"]:
-             path = os.path.join(os.path.dirname(__file__), base)
-             if os.path.isdir(path):
-                 shutil.rmtree(path)
+            path = os.path.join(os.path.dirname(__file__), base)
+            if os.path.isdir(path):
+                shutil.rmtree(path)
+
 
 class coverage_cmd(Command):
     description = "generate test coverage data"
@@ -41,7 +48,7 @@ class coverage_cmd(Command):
 
     def initialize_options(self):
         pass
-    
+
     def finalize_options(self):
         pass
 
@@ -50,6 +57,7 @@ class coverage_cmd(Command):
         tracer = trace.Trace(
             count=True, trace=False,
             ignoredirs=[sys.prefix, sys.exec_prefix])
+
         def run_tests():
             import bulletml
             try:
@@ -57,13 +65,16 @@ class coverage_cmd(Command):
             except NameError:
                 pass
             self.run_command("test")
+
         tracer.runfunc(run_tests)
         results = tracer.results()
         coverage = os.path.join(os.path.dirname(__file__), "coverage")
         results.write_results(show_missing=True, coverdir=coverage)
         map(os.unlink, glob.glob(os.path.join(coverage, "[!b]*.cover")))
-        try: os.unlink(os.path.join(coverage, "..setup.cover"))
-        except OSError: pass
+        try:
+            os.unlink(os.path.join(coverage, "..setup.cover"))
+        except OSError:
+            pass
 
         total_lines = 0
         bad_lines = 0
@@ -78,16 +89,18 @@ class coverage_cmd(Command):
         print("Coverage data written to %s (%d/%d, %0.2f%%)" % (
             coverage, total_lines - bad_lines, total_lines, pct))
 
+
 class sdist(distutils_sdist):
     def run(self):
         self.run_command("test")
         distutils_sdist.run(self)
 
+
 class test_cmd(Command):
     description = "run automated tests"
     user_options = [
         ("to-run=", None, "list of tests to run (default all)"),
-        ]
+    ]
 
     def initialize_options(self):
         self.to_run = []