kuuko pushed a commit to branch master.

http://git.enlightenment.org/bindings/python/python-efl.git/commit/?id=fdfa113926756f17551d75bc9ead87fb570480c2

commit fdfa113926756f17551d75bc9ead87fb570480c2
Author: Kai Huuhko <kai.huu...@gmail.com>
Date:   Tue Apr 29 12:20:33 2014 +0300

    setup.py: pep-8 cleanup
---
 setup.py | 311 ++++++++++++++++++++++++++++++++++++---------------------------
 1 file changed, 177 insertions(+), 134 deletions(-)

diff --git a/setup.py b/setup.py
index 897dcf4..44ee4ec 100755
--- a/setup.py
+++ b/setup.py
@@ -32,11 +32,18 @@ try:
     from sphinx.setup_command import BuildDoc
 except ImportError:
     class BuildDoc(Command):
-        description = "build documentation using sphinx, that must be 
installed."
+        description = \
+            "build documentation using sphinx, that must be installed."
         user_options = []
-        def initialize_options(self): pass
-        def finalize_options(self): pass
-        def run(self): print("Error: sphinx not found")
+
+        def initialize_options(self):
+            pass
+
+        def finalize_options(self):
+            pass
+
+        def run(self):
+            print("Error: sphinx not found")
 
 
 # === pkg-config ===
@@ -71,7 +78,9 @@ def pkg_config(name, require, min_vers=None):
     except (OSError, subprocess.CalledProcessError):
         raise SystemExit("Did not find " + name + " with 'pkg-config'.")
     except (AssertionError):
-        raise SystemExit(name+" version mismatch. Found: "+ver+"  Needed: 
"+min_vers)
+        raise SystemExit(
+            name + " version mismatch. Found: " + ver + "  Needed: " + min_vers
+        )
 
 
 # use cython or pre-generated c files
@@ -84,31 +93,41 @@ if os.path.exists(os.path.join(script_path, "efl", "eo", 
"efl.eo.pyx")):
         from Cython.Build import cythonize
         import Cython.Compiler.Options
     except ImportError:
-        raise SystemExit("Requires Cython >= %s (http://cython.org/)" % 
CYTHON_MIN_VERSION)
+        raise SystemExit(
+            "Requires Cython >= %s (http://cython.org/)" % CYTHON_MIN_VERSION
+        )
 
     try:
         try:
-            assert StrictVersion(Cython.__version__) >= 
StrictVersion(CYTHON_MIN_VERSION)
+            assert StrictVersion(Cython.__version__) >= \
+                StrictVersion(CYTHON_MIN_VERSION)
         except ValueError:
-            print("Your Cython version string (%s) is weird. We'll attempt to "
-                "check that it's higher than the minimum required: %s, but 
this "
-                "is unreliable.\n"
-                "If you run into any problems during or after installation it 
may "
-                "be caused by version of Cython that's too old." % (
+            print(
+                "Your Cython version string (%s) is weird. We'll attempt to "
+                "check that it's higher than the minimum required: %s, but "
+                "this is unreliable.\n"
+                "If you run into any problems during or after installation it "
+                "may be caused by version of Cython that's too old." % (
                     Cython.__version__, CYTHON_MIN_VERSION
-                    )
                 )
-            assert LooseVersion(Cython.__version__) >= 
LooseVersion(CYTHON_MIN_VERSION)
+            )
+            assert LooseVersion(Cython.__version__) >= \
+                LooseVersion(CYTHON_MIN_VERSION)
     except AssertionError:
-        raise SystemExit("Requires Cython >= %s (http://cython.org/)" % 
CYTHON_MIN_VERSION)
+        raise SystemExit(
+            "Requires Cython >= %s (http://cython.org/)" % CYTHON_MIN_VERSION
+        )
 
-    Cython.Compiler.Options.fast_fail = True # Stop compilation on first error
-    Cython.Compiler.Options.annotate = False # Generates HTML files with 
annotated source
-    Cython.Compiler.Options.docstrings = True # Set to False to disable 
docstrings
+    Cython.Compiler.Options.fast_fail = True  # Stop compilation on first error
+    Cython.Compiler.Options.annotate = False  # Generates HTML files with
+                                              #  annotated source
+    Cython.Compiler.Options.docstrings = True  # Set to False to disable
+                                               #  docstrings
 
 else:
     module_suffix = ".c"
     from distutils.command.build_ext import build_ext
+
     def cythonize(modules, *args, **kwargs):
         return modules
 
@@ -116,16 +135,24 @@ else:
 class CleanGenerated(Command):
     description = "Clean C and html files generated by Cython"
     user_options = []
-    def initialize_options(self): pass
-    def finalize_options(self): pass
+
+    def initialize_options(self):
+        pass
+
+    def finalize_options(self):
+        pass
+
     def run(self):
-        for lib in "eo", "evas", "ecore", "edje", "edje/edit", "emotion", 
"elementary", "utils":
-            for root, dirs, files in os.walk(os.path.join(script_path, "efl", 
lib)):
+        for lib in ("eo", "evas", "ecore", "edje", "edje/edit", "emotion",
+                    "elementary", "utils"):
+            for root, dirs, files in \
+                    os.walk(os.path.join(script_path, "efl", lib)):
                 for f in files:
                     if f.endswith(".c") or f.endswith(".html"):
                         path = os.path.join(root, f)
                         os.remove(path)
-        dbus_ml_path = os.path.join(script_path, "efl", "dbus_mainloop", 
"dbus_mainloop.c")
+        dbus_ml_path = os.path.join(
+            script_path, "efl", "dbus_mainloop", "dbus_mainloop.c")
         if os.path.exists(dbus_ml_path):
             os.remove(dbus_ml_path)
 
@@ -139,94 +166,100 @@ if set(("build", "build_ext", "install", "bdist", 
"sdist")) & set(sys.argv):
     # === Eina ===
     eina_cflags, eina_libs = pkg_config('Eina', 'eina', EFL_MIN_VERSION)
 
-
     # === Eo ===
     eo_cflags, eo_libs = pkg_config('Eo', 'eo', EFL_MIN_VERSION)
-    eo_ext = Extension("eo", ["efl/eo/efl.eo"+module_suffix],
-                       define_macros = [('EFL_BETA_API_SUPPORT', None)],
-                       include_dirs = ['include/'],
-                       extra_compile_args = eo_cflags,
-                       extra_link_args = eo_libs + eina_libs
-                      )
+    eo_ext = Extension(
+        "eo", ["efl/eo/efl.eo" + module_suffix],
+        define_macros=[('EFL_BETA_API_SUPPORT', None)],
+        include_dirs=['include/'],
+        extra_compile_args=eo_cflags,
+        extra_link_args=eo_libs + eina_libs
+    )
     modules.append(eo_ext)
 
-
     # === Utilities ===
     utils_ext = [
-        Extension("utils.deprecated", ["efl/utils/deprecated"+module_suffix],
-            include_dirs = ['include/'],
-            extra_compile_args = eina_cflags,
-            extra_link_args = eina_libs
-            ),
-        Extension("utils.conversions", ["efl/utils/conversions"+module_suffix],
-            include_dirs = ['include/'],
-            extra_compile_args = eo_cflags,
-            extra_link_args = eo_libs + eina_libs,
-            ),
-        Extension("utils.logger", ["efl/utils/logger"+module_suffix],
-            include_dirs = ['include/'],
-            extra_compile_args = eina_cflags,
-            extra_link_args = eina_libs,
-            ),
-        ]
+        Extension(
+            "utils.deprecated", ["efl/utils/deprecated" + module_suffix],
+            include_dirs=['include/'],
+            extra_compile_args=eina_cflags,
+            extra_link_args=eina_libs
+        ),
+        Extension(
+            "utils.conversions", ["efl/utils/conversions" + module_suffix],
+            include_dirs=['include/'],
+            extra_compile_args=eo_cflags,
+            extra_link_args=eo_libs + eina_libs,
+        ),
+        Extension(
+            "utils.logger", ["efl/utils/logger" + module_suffix],
+            include_dirs=['include/'],
+            extra_compile_args=eina_cflags,
+            extra_link_args=eina_libs,
+        ),
+    ]
     modules += utils_ext
     packages.append("efl.utils")
 
-
     # === Evas ===
     evas_cflags, evas_libs = pkg_config('Evas', 'evas', EFL_MIN_VERSION)
-    evas_ext = Extension("evas", ["efl/evas/efl.evas"+module_suffix],
-                         include_dirs = ['include/'],
-                         extra_compile_args = evas_cflags,
-                         extra_link_args = evas_libs + eina_libs,
-                        )
+    evas_ext = Extension(
+        "evas", ["efl/evas/efl.evas" + module_suffix],
+        include_dirs=['include/'],
+        extra_compile_args=evas_cflags,
+        extra_link_args=evas_libs + eina_libs,
+    )
     modules.append(evas_ext)
 
     # compatibility
     packages.append("evas")
     package_dirs["evas"] = "compat/evas"
 
-
     # === Ecore ===
     ecore_cflags, ecore_libs = pkg_config('Ecore', 'ecore', EFL_MIN_VERSION)
-    ecore_file_cflags, ecore_file_libs = pkg_config('EcoreFile', 'ecore-file', 
"1.8.0")
-    ecore_ext = Extension("ecore", ["efl/ecore/efl.ecore"+module_suffix],
-                          include_dirs = ['include/'],
-                          extra_compile_args = list(set(ecore_cflags + 
ecore_file_cflags)),
-                          extra_link_args = ecore_libs + ecore_file_libs + 
eina_libs + evas_libs,
-                         )
+    ecore_file_cflags, ecore_file_libs = pkg_config(
+        'EcoreFile', 'ecore-file', EFL_MIN_VERSION)
+    ecore_ext = Extension(
+        "ecore", ["efl/ecore/efl.ecore" + module_suffix],
+        include_dirs=['include/'],
+        extra_compile_args=list(set(ecore_cflags + ecore_file_cflags)),
+        extra_link_args=ecore_libs + ecore_file_libs + eina_libs + evas_libs,
+    )
     modules.append(ecore_ext)
 
     # compatibility
     packages.append("ecore")
     package_dirs["ecore"] = "compat/ecore"
 
-
     # === Ethumb ===
-    ethumb_cflags, ethumb_libs = pkg_config('Ethumb', 'ethumb', 
EFL_MIN_VERSION)
-    ethumb_ext = Extension("ethumb", ["efl/ethumb/efl.ethumb"+module_suffix],
-                         include_dirs = ['include/'],
-                         extra_compile_args = ethumb_cflags,
-                         extra_link_args = ethumb_libs + eina_libs,
-                        )
+    ethumb_cflags, ethumb_libs = pkg_config(
+        'Ethumb', 'ethumb', EFL_MIN_VERSION)
+    ethumb_ext = Extension(
+        "ethumb", ["efl/ethumb/efl.ethumb" + module_suffix],
+        include_dirs=['include/'],
+        extra_compile_args=ethumb_cflags,
+        extra_link_args=ethumb_libs + eina_libs,
+    )
     modules.append(ethumb_ext)
 
-    ethumb_client_cflags, ethumb_client_libs = pkg_config('Ethumb_Client', 
'ethumb_client', EFL_MIN_VERSION)
-    ethumb_client_ext = Extension("ethumb_client", 
["efl/ethumb/efl.ethumb_client"+module_suffix],
-                         include_dirs = ['include/'],
-                         extra_compile_args = ethumb_client_cflags,
-                         extra_link_args = ethumb_client_libs + eina_libs,
-                        )
+    ethumb_client_cflags, ethumb_client_libs = pkg_config(
+        'Ethumb_Client', 'ethumb_client', EFL_MIN_VERSION)
+    ethumb_client_ext = Extension(
+        "ethumb_client", ["efl/ethumb/efl.ethumb_client" + module_suffix],
+        include_dirs=['include/'],
+        extra_compile_args=ethumb_client_cflags,
+        extra_link_args=ethumb_client_libs + eina_libs,
+    )
     modules.append(ethumb_client_ext)
 
-
     # === Edje ===
     edje_cflags, edje_libs = pkg_config('Edje', 'edje', EFL_MIN_VERSION)
-    edje_ext = Extension("edje", ["efl/edje/efl.edje"+module_suffix],
-                         include_dirs = ['include/'],
-                         extra_compile_args = edje_cflags,
-                         extra_link_args = edje_libs + eina_libs + evas_libs,
-                        )
+    edje_ext = Extension(
+        "edje", ["efl/edje/efl.edje" + module_suffix],
+        include_dirs=['include/'],
+        extra_compile_args=edje_cflags,
+        extra_link_args=edje_libs + eina_libs + evas_libs,
+    )
     modules.append(edje_ext)
 
     # compatibility
@@ -234,44 +267,46 @@ if set(("build", "build_ext", "install", "bdist", 
"sdist")) & set(sys.argv):
     package_dirs["edje"] = "compat/edje"
 
     # --- Edje_Edit ---
-    edje_edit_ext = Extension("edje_edit", 
["efl/edje/efl.edje_edit"+module_suffix],
-                              define_macros = 
[('EDJE_EDIT_IS_UNSTABLE_AND_I_KNOW_ABOUT_IT', None)],
-                              include_dirs = ['include/'],
-                              extra_compile_args = edje_cflags,
-                              extra_link_args = edje_libs + eina_libs + 
evas_libs,
-                             )
+    edje_edit_ext = Extension(
+        "edje_edit", ["efl/edje/efl.edje_edit" + module_suffix],
+        define_macros=[('EDJE_EDIT_IS_UNSTABLE_AND_I_KNOW_ABOUT_IT', None)],
+        include_dirs=['include/'],
+        extra_compile_args=edje_cflags,
+        extra_link_args=edje_libs + eina_libs + evas_libs,
+    )
     modules.append(edje_edit_ext)
 
-
     # === Emotion ===
-    emotion_cflags, emotion_libs = pkg_config('Emotion', 'emotion', 
EFL_MIN_VERSION)
-    emotion_ext = Extension("emotion", 
["efl/emotion/efl.emotion"+module_suffix],
-                            include_dirs = ['include/'],
-                            extra_compile_args = emotion_cflags,
-                            extra_link_args = emotion_libs + eina_libs + 
evas_libs,
-                           )
+    emotion_cflags, emotion_libs = pkg_config(
+        'Emotion', 'emotion', EFL_MIN_VERSION)
+    emotion_ext = Extension(
+        "emotion", ["efl/emotion/efl.emotion" + module_suffix],
+        include_dirs=['include/'],
+        extra_compile_args=emotion_cflags,
+        extra_link_args=emotion_libs +
+        eina_libs + evas_libs,
+    )
     modules.append(emotion_ext)
 
     # compatibility
     packages.append("emotion")
     package_dirs["emotion"] = "compat/emotion"
 
-
     # === dbus mainloop integration ===
     dbus_cflags, dbus_libs = pkg_config('DBus', 'dbus-python', "0.83.0")
-    dbus_ml_ext = Extension("dbus_mainloop",
-                            ["efl/dbus_mainloop/dbus_mainloop"+module_suffix,
-                             "efl/dbus_mainloop/e_dbus.c"],
-                            extra_compile_args = list(set(dbus_cflags + 
ecore_cflags)),
-                            extra_link_args = dbus_libs + ecore_libs,
-                           )
+    dbus_ml_ext = Extension(
+        "dbus_mainloop",
+        ["efl/dbus_mainloop/dbus_mainloop" + module_suffix,
+         "efl/dbus_mainloop/e_dbus.c"],
+        extra_compile_args=list(set(dbus_cflags + ecore_cflags)),
+        extra_link_args=dbus_libs + ecore_libs,
+        )
     modules.append(dbus_ml_ext)
 
     # compatibility
     packages.append("e_dbus")
     package_dirs["e_dbus"] = "compat/e_dbus"
 
-
     # === Elementary ===
     elm_mods = (
         #"access",
@@ -347,14 +382,16 @@ if set(("build", "build_ext", "install", "bdist", 
"sdist")) & set(sys.argv):
         "window",
     )
 
-    elm_cflags, elm_libs = pkg_config('Elementary', 'elementary', 
ELM_MIN_VERSION)
+    elm_cflags, elm_libs = pkg_config(
+        'Elementary', 'elementary', ELM_MIN_VERSION)
     for m in elm_mods:
-        e = Extension("elementary." + m,
-                      ["efl/elementary/" + m + module_suffix],
-                      include_dirs = ["include/"],
-                      extra_compile_args = elm_cflags,
-                      extra_link_args = elm_libs + eina_libs + evas_libs,
-                     )
+        e = Extension(
+            "elementary." + m,
+            ["efl/elementary/" + m + module_suffix],
+            include_dirs=["include/"],
+            extra_compile_args=elm_cflags,
+            extra_link_args=elm_libs + eina_libs + evas_libs,
+            )
         modules.append(e)
 
     packages.append("efl.elementary")
@@ -365,37 +402,43 @@ if set(("build", "build_ext", "install", "bdist", 
"sdist")) & set(sys.argv):
 
 
 setup(
-    name = "python-efl",
-    fullname = "Python bindings for Enlightenment Foundation Libraries",
-    description = "Python bindings for Enlightenment Foundation Libraries",
-    version = RELEASE,
-    author = "Gustavo Sverzut Barbieri, Simon Busch, Boris 'billiob' Faure, 
Davide Andreoli, Fabiano Fidêncio, Bruno Dilly, Tiago Falcão, Joost Albers, Kai 
Huuhko, Ulisses Furquim",
-    author_email = "d...@gurumeditation.it, kai.huu...@gmail.com",
-    maintainer = "Kai Huuhko, Davide Andreoli",
-    maintainer_email = "kai.huu...@gmail.com, d...@gurumeditation.it",
-    contact = "Enlightenment developer mailing list",
-    contact_email = "enlightenment-devel@lists.sourceforge.net",
-    url = "http://www.enlightenment.org";,
-    license = "GNU Lesser General Public License (LGPL)",
-    cmdclass = {
+    name="python-efl",
+    fullname="Python bindings for Enlightenment Foundation Libraries",
+    description="Python bindings for Enlightenment Foundation Libraries",
+    version=RELEASE,
+    author=(
+        "Gustavo Sverzut Barbieri, Simon Busch, Boris 'billiob' Faure, "
+        "Davide Andreoli, Fabiano Fidêncio, Bruno Dilly, Tiago Falcão, "
+        "Joost Albers, Kai Huuhko, Ulisses Furquim"
+        ),
+    author_email="d...@gurumeditation.it, kai.huu...@gmail.com",
+    maintainer="Kai Huuhko, Davide Andreoli",
+    maintainer_email="kai.huu...@gmail.com, d...@gurumeditation.it",
+    contact="Enlightenment developer mailing list",
+    contact_email="enlightenment-devel@lists.sourceforge.net",
+    url="http://www.enlightenment.org";,
+    license="GNU Lesser General Public License (LGPL)",
+    cmdclass={
         'build_ext': build_ext,
         'build_doc': BuildDoc,
         'clean_generated_files': CleanGenerated
-        },
-    command_options = {
+    },
+    command_options={
         'build_doc': {
             'version': ('setup.py', VERSION),
             'release': ('setup.py', RELEASE)
-            }
-        },
-    package_dir = package_dirs,
-    packages = packages,
-    ext_package = "efl", # The prefix for ext modules/packages
-    ext_modules = cythonize(modules, include_path=["include"],
-                            compiler_directives={
-                                #"c_string_type": "unicode",
-                                #"c_string_encoding": "utf-8",
-                                "embedsignature": True,
-                                }
-                            ),
+        }
+    },
+    package_dir=package_dirs,
+    packages=packages,
+    ext_package="efl",  # The prefix for ext modules/packages
+    ext_modules=cythonize(
+        modules,
+        include_path=["include"],
+        compiler_directives={
+            #"c_string_type": "unicode",
+            #"c_string_encoding": "utf-8",
+            "embedsignature": True,
+        }
+    ),
 )

-- 


Reply via email to