Skip to content

Commit

Permalink
Allow older ConfigArgParse for users of modern Pythons
Browse files Browse the repository at this point in the history
(I think this is a bad idea because of
pypa/pip#3025, but letsencrypt maintainers
insist, so *shrug*.  Also the same problem exists for the versioned
'mock' dependency, so I'm not introducing a new one here.)
  • Loading branch information
mgedmin committed Dec 3, 2015
1 parent 9fbec03 commit 51f1711
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@ def read_file(filename, encoding='utf8'):

install_requires = [
'acme=={0}'.format(version),
'ConfigArgParse>=0.10.0', # python2.6 support, upstream #17
'configobj',
'cryptography>=0.7', # load_pem_x509_certificate
'parsedatetime',
Expand All @@ -54,9 +53,13 @@ def read_file(filename, encoding='utf8'):
# only some distros recognize stdlib argparse as already satisfying
'argparse',
'mock<1.1.0',
'ConfigArgParse>=0.10.0', # python2.6 support, upstream #17
])
else:
install_requires.append('mock')
install_requires.extend([
'mock',
'ConfigArgParse',

This comment has been minimized.

Copy link
@kuba

kuba Dec 3, 2015

we order imports :)

This comment has been minimized.

Copy link
@mgedmin

mgedmin Dec 3, 2015

Author Owner

Alphabetically, I assume ;)

Also, judging from the imports above, the ordering is case-insensitive.

I'll update.

])

dev_extras = [
# Pin astroid==1.3.5, pylint==1.4.2 as a workaround for #289
Expand Down

0 comments on commit 51f1711

Please sign in to comment.