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

Install six before we upgrade setuptools. #5541

Merged
merged 7 commits into from
Jun 1, 2017

Conversation

EnTeQuAk
Copy link
Contributor

@EnTeQuAk EnTeQuAk commented Jun 1, 2017

Setuptools depends on six but apparently doesn't install it as a dependency so we need to do that, at least until there's an upstream fix.

See pypa/setuptools#964 for a few more details. Currently quite a few
travis and circleci jobs are failing because of that.

Notes:

  • This now installs and upgrades six, setuptools and pip everywhere, without exceptions
  • That requires now more docs requirements since require-hashes mode now functions for docs too

See pypa/setuptools#964 for a few more details. Currently quite a few
travis and circleci jobs are failing because of that.
@@ -31,7 +31,9 @@ RUN yum install -y \
uwsgi-2.0.13.1-2.el7 \
uwsgi-plugin-python \
&& yum clean all \
&& curl -sSL https://bootstrap.pypa.io/get-pip.py | python
&& curl -sSL https://bootstrap.pypa.io/get-pip.py | python \
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ugh, can't we replace that with yum install -y python-pip as well ? /cc @bqbn

@EnTeQuAk EnTeQuAk merged commit e3e020d into master Jun 1, 2017
@EnTeQuAk EnTeQuAk deleted the install-six-before-upgradingsetuptools branch June 1, 2017 11:20
@diox
Copy link
Member

diox commented Jun 1, 2017

FWIW the issue was fixed in setuptools 36.0.1, which I merged in #5545

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

Successfully merging this pull request may close these issues.

2 participants