Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

setuptools being downgraded #1428

Closed
rbtcollins opened this issue Jul 12, 2015 · 3 comments
Closed

setuptools being downgraded #1428

rbtcollins opened this issue Jul 12, 2015 · 3 comments

Comments

@rbtcollins
Copy link

This is a follow-on to #1426. With the newer virtualenv the requirements.txt processing does manage to complete, but the install target fails due to very old setuptools (1.1!) - the doc_builder target appears to be downgrading it to 1.1 in the middle. :(.

doc_builder
-----

Collecting sphinx==1.3.1
...
Collecting virtualenv==1.10.1
Collecting setuptools==1.1
...
  Found existing installation: setuptools 18.0.1
    Uninstalling setuptools-18.0.1:
      Successfully uninstalled setuptools-18.0.1
Successfully installed Jinja2-2.7.3 Markdown-2.6.2 PyYAML-3.11 Pygments-2.0.2 alabaster-0.7.6 babel-1.3 backports.ssl-match-hostname-3.4.0.2 certifi-2015.4.28 click-4.0 commonmark-0.5.4 docutils-0.11 livereload-2.4.0 markupsafe-0.23 mkdocs-0.14.0 mock-1.0.1 nilsimsa-0.3.7 pillow-2.6.1 pytz-2015.4 readthedocs-sphinx-ext-0.5.4 recommonmark-0.1.1 requests-2.7.0 setuptools-1.1 six-1.9.0 snowballstemmer-1.2.0 sphinx-1.3.1 sphinx-rtd-theme-0.1.8 tornado-4.2 virtualenv-1.10.1

This is from https://readthedocs.org/builds/mock/3074098/

@rbtcollins
Copy link
Author

And https://github.com/rtfd/readthedocs.org/blob/b4c6c7bcd750bd0751c7a151b485b22df932a435/readthedocs/projects/tasks.py#L335 seems to be the culprit. I don't know why setuptools 1.1 is being pinned - but its really rather old. In fact - d5d71c0 is where it was introduced.

Would a PR to bump that to 18 be acceptable?

gregmuellegger added a commit that referenced this issue Jul 13, 2015
We updated the virtualenv in requirements/pip.txt for #1426. However that
did only touch the virtualenv of readthedocs, not the ones of the builds. So
updating it with this change.

Fixes #1428.
@gregmuellegger
Copy link
Contributor

I created #1429 as a fix. I want to get it reviewed before we can deploy it. If you need a very quick workaround you can pin the setuptools/virtualenv versions you need in your repo's requirements.txt.

@ericholscher
Copy link
Member

Should be deployed.

On Mon, Jul 13, 2015 at 12:45 AM, Gregor Müllegger <[email protected]

wrote:

I created #1429 #1429 as a
fix. I want to get it reviewed before we can deploy it. If you need a very
quick workaround you can pin the setuptools/virtualenv versions you need in
your repo's requirements.txt.


Reply to this email directly or view it on GitHub
#1428 (comment)
.

Eric Holscher
Maker of the internet residing in Portland, Oregon
http://ericholscher.com

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants