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 Python dependencies without sudo #3894

Merged
merged 1 commit into from Jul 10, 2015
Merged

Install Python dependencies without sudo #3894

merged 1 commit into from Jul 10, 2015

Conversation

tschaub
Copy link
Member

@tschaub tschaub commented Jul 10, 2015

This should allow us to use the container based infrastructure on Travis.

Fixes #3893.

@elemoine
Copy link
Member

Cool !

@tschaub
Copy link
Member Author

tschaub commented Jul 10, 2015

Hmm. Thought we'd get a virtualenv's Python by default, but it looks like this is still trying to install things system-wide.

@elemoine any ideas?

@tschaub
Copy link
Member Author

tschaub commented Jul 10, 2015

Seeing if things work if we force a Python VM image.

@tschaub
Copy link
Member Author

tschaub commented Jul 10, 2015

OK, it looks like we're successfully running on Travis CI's container infrastructure (compare an old build with one from this branch).

I was hoping this would mean faster build time, but it doesn't look like a significant difference. It should at least mean a faster startup time (but I haven't compared with an old build).

@elemoine
Copy link
Member

Yep, language: python need be used so pip installs in a virtual env.

@elemoine
Copy link
Member

LGTM. Thanks!

tschaub added a commit that referenced this pull request Jul 10, 2015
Install Python dependencies without sudo.
@tschaub tschaub merged commit 8600d46 into openlayers:master Jul 10, 2015
@tschaub tschaub deleted the no-sudo branch July 10, 2015 20:20
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.

None yet

2 participants