Skip to content

Commit

Permalink
Fix PEP8 compatibility of setup.py.
Browse files Browse the repository at this point in the history
  • Loading branch information
jelmer committed May 2, 2013
1 parent dda8ad5 commit a186cc5
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 11 deletions.
8 changes: 4 additions & 4 deletions dulwich/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ class Config(object):

def get(self, section, name):
"""Retrieve the contents of a configuration setting.
:param section: Tuple with section name and optional subsection namee
:param subsection: Subsection name
:return: Contents of the setting
Expand Down Expand Up @@ -67,7 +67,7 @@ def get_boolean(self, section, name, default=None):

def set(self, section, name, value):
"""Set a configuration value.
:param name: Name of the configuration value, including section
and optional subsection
:param: Value of the setting
Expand All @@ -94,10 +94,10 @@ def __eq__(self, other):

def __getitem__(self, key):
return self._values[key]

def __setitem__(self, key, value):
self._values[key] = value

def keys(self):
return self._values.keys()

Expand Down
14 changes: 7 additions & 7 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,8 @@ def has_ext_modules(self):
return not self.pure and not '__pypy__' in sys.modules

global_options = Distribution.global_options + [
('pure', None,
"use pure Python code instead of C extensions (slower on CPython)")]
('pure', None, "use pure Python code instead of C "
"extensions (slower on CPython)")]

pure = False

Expand All @@ -45,8 +45,7 @@ def has_ext_modules(self):
out, err = p.communicate()
for l in out.splitlines():
# Also parse only first digit, because 3.2.1 can't be parsed nicely
if (l.startswith('Xcode') and
int(l.split()[1].split('.')[0]) >= 4):
if l.startswith('Xcode') and int(l.split()[1].split('.')[0]) >= 4:
os.environ['ARCHFLAGS'] = ''

setup_kwargs = {}
Expand All @@ -59,7 +58,8 @@ def has_ext_modules(self):
keywords='git',
version=dulwich_version_string,
url='http://samba.org/~jelmer/dulwich',
download_url='http://samba.org/~jelmer/dulwich/dulwich-%s.tar.gz' % dulwich_version_string,
download_url='http://samba.org/~jelmer/dulwich/'
'dulwich-%s.tar.gz' % dulwich_version_string,
license='GPLv2 or later',
author='Jelmer Vernooij',
author_email='jelmer@samba.org',
Expand All @@ -73,14 +73,14 @@ def has_ext_modules(self):
""",
packages=['dulwich', 'dulwich.tests'],
scripts=['bin/dulwich', 'bin/dul-daemon', 'bin/dul-web'],
ext_modules = [
ext_modules=[
Extension('dulwich._objects', ['dulwich/_objects.c'],
include_dirs=include_dirs),
Extension('dulwich._pack', ['dulwich/_pack.c'],
include_dirs=include_dirs),
Extension('dulwich._diff_tree', ['dulwich/_diff_tree.c'],
include_dirs=include_dirs),
],
],
distclass=DulwichDistribution,
**setup_kwargs
)

0 comments on commit a186cc5

Please sign in to comment.