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

Make use of the new travis-ci caching mechanism. #327

Merged
merged 2 commits into from May 29, 2015
Merged

Make use of the new travis-ci caching mechanism. #327

merged 2 commits into from May 29, 2015

Conversation

hannosch
Copy link
Contributor

This moves the Zookeeper install from bin/zookeeper-3.4.6 to zookeeper/3.4.6 to make it easier to cache it.

In the new sudo-less environment you can't use sudo apt-get anymore. This might result in failing to installed gevent 0.13.8, gevent 1.0.x includes a copy of libev inside the tar.gz, so it should build fine.

First test to see if the caches get populated...

@hannosch
Copy link
Contributor Author

Got libevent-dev installed again via the apt addon.

The build and cache are working, there are just individual test failures as before. The cache itself doesn't seem to help in terms of test runtime, though it does make the build less dependent on the Apache mirror for getting the ZK archive.

Since the old sudo infrastructure is going away at some point, it's probably still worth it to switch and merge this.

@harlowja
Copy link
Contributor

Looks good to me, although I've been unable to get to figure out the cause of travis still messing up (with nearly always noautherrors); can't seem to cause it to happen locally so unsure whats up with that :-P

bbangert added a commit that referenced this pull request May 29, 2015
Make use of the new travis-ci caching mechanism.
@bbangert bbangert merged commit 14480e6 into python-zk:master May 29, 2015
@hannosch hannosch deleted the travis-cache branch May 29, 2015 08:27
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

3 participants