Skip to content

Commit

Permalink
Merge branch 'testing-dev-extras'
Browse files Browse the repository at this point in the history
  • Loading branch information
bmw committed Feb 13, 2016
2 parents e493a0d + 86d6d27 commit 45ee1eb
Show file tree
Hide file tree
Showing 7 changed files with 23 additions and 29 deletions.
2 changes: 1 addition & 1 deletion Dockerfile-dev
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ RUN virtualenv --no-site-packages -p python2 /opt/letsencrypt/venv && \
-e /opt/letsencrypt/src/letsencrypt-nginx \
-e /opt/letsencrypt/src/letshelp-letsencrypt \
-e /opt/letsencrypt/src/letsencrypt-compatibility-test \
-e /opt/letsencrypt/src[dev,docs,testing]
-e /opt/letsencrypt/src[dev,docs]

# install in editable mode (-e) to save space: it's not possible to
# "rm -rf /opt/letsencrypt/src" (it's stays in the underlaying image);
Expand Down
13 changes: 7 additions & 6 deletions acme/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,17 +34,18 @@
else:
install_requires.append('mock')

dev_extras = [
'nose',
'pep8',
'tox',
]

docs_extras = [
'Sphinx>=1.0', # autodoc_member_order = 'bysource', autodoc_default_flags
'sphinx_rtd_theme',
'sphinxcontrib-programoutput',
]

testing_extras = [
'nose',
'tox',
]


setup(
name='acme',
Expand Down Expand Up @@ -74,8 +75,8 @@
include_package_data=True,
install_requires=install_requires,
extras_require={
'dev': dev_extras,
'docs': docs_extras,
'testing': testing_extras,
},
entry_points={
'console_scripts': [
Expand Down
3 changes: 0 additions & 3 deletions setup.cfg
Original file line number Diff line number Diff line change
@@ -1,9 +1,6 @@
[easy_install]
zip_ok = false

[aliases]
dev = develop easy_install letsencrypt[dev,docs,testing]

[nosetests]
nocapture=1
cover-package=letsencrypt,acme,letsencrypt_apache,letsencrypt_nginx
Expand Down
14 changes: 5 additions & 9 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,12 @@ def read_file(filename, encoding='utf8'):
dev_extras = [
# Pin astroid==1.3.5, pylint==1.4.2 as a workaround for #289
'astroid==1.3.5',
'coverage',
'nose',
'nosexcover',
'pep8',
'pylint==1.4.2', # upstream #248
'tox',
'twine',
'wheel',
]
Expand All @@ -77,14 +82,6 @@ def read_file(filename, encoding='utf8'):
'sphinxcontrib-programoutput',
]

testing_extras = [
'coverage',
'nose',
'nosexcover',
'pep8',
'tox',
]

setup(
name='letsencrypt',
version=version,
Expand Down Expand Up @@ -120,7 +117,6 @@ def read_file(filename, encoding='utf8'):
extras_require={
'dev': dev_extras,
'docs': docs_extras,
'testing': testing_extras,
},

# to test all packages run "python setup.py test -s
Expand Down
4 changes: 2 additions & 2 deletions tools/venv.sh
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,8 @@
export VENV_ARGS="--python python2"

./tools/_venv_common.sh \
-e acme[testing] \
-e .[dev,docs,testing] \
-e acme[dev] \
-e .[dev,docs] \
-e letsencrypt-apache \
-e letsencrypt-nginx \
-e letshelp-letsencrypt \
Expand Down
2 changes: 1 addition & 1 deletion tools/venv3.sh
Original file line number Diff line number Diff line change
Expand Up @@ -5,4 +5,4 @@ export VENV_NAME="${VENV_NAME:-venv3}"
export VENV_ARGS="--python python3"

./tools/_venv_common.sh \
-e acme[testing] \
-e acme[dev] \
14 changes: 7 additions & 7 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,9 @@ envlist = py{26,27,33,34,35},py{26,27}-oldest,cover,lint
# packages installed separately to ensure that downstream deps problems
# are detected, c.f. #1002
commands =
pip install -e acme[testing]
pip install -e acme[dev]
nosetests -v acme
pip install -e .[testing]
pip install -e .[dev]
nosetests -v letsencrypt
pip install -e letsencrypt-apache
nosetests -v letsencrypt_apache
Expand All @@ -40,23 +40,23 @@ deps =

[testenv:py33]
commands =
pip install -e acme[testing]
pip install -e acme[dev]
nosetests -v acme

[testenv:py34]
commands =
pip install -e acme[testing]
pip install -e acme[dev]
nosetests -v acme

[testenv:py35]
commands =
pip install -e acme[testing]
pip install -e acme[dev]
nosetests -v acme

[testenv:cover]
basepython = python2.7
commands =
pip install -e acme -e .[testing] -e letsencrypt-apache -e letsencrypt-nginx -e letshelp-letsencrypt
pip install -e acme[dev] -e .[dev] -e letsencrypt-apache -e letsencrypt-nginx -e letshelp-letsencrypt
./tox.cover.sh

[testenv:lint]
Expand All @@ -66,7 +66,7 @@ basepython = python2.7
# duplicate code checking; if one of the commands fails, others will
# continue, but tox return code will reflect previous error
commands =
pip install -e acme -e .[dev] -e letsencrypt-apache -e letsencrypt-nginx -e letsencrypt-compatibility-test -e letshelp-letsencrypt
pip install -e acme[dev] -e .[dev] -e letsencrypt-apache -e letsencrypt-nginx -e letsencrypt-compatibility-test -e letshelp-letsencrypt
./pep8.travis.sh
pylint --rcfile=.pylintrc letsencrypt
pylint --rcfile=acme/.pylintrc acme/acme
Expand Down

0 comments on commit 45ee1eb

Please sign in to comment.