-
Notifications
You must be signed in to change notification settings - Fork 715
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
Improve Travis CI Android Builds #548
Conversation
Can't find anything malicious in the changes. |
Thanks a ton for this! I rebased these changes and pushed them (otherwise unmodified) to master: 52c645b Don't install unnecessary packages on Travis CI Android builds. |
Cool! Hopefully next week I'll have time to debug the 32-bit ARM issues when using clang. |
@pietro After this change some builds are failing with "No space left on device": https://travis-ci.org/briansmith/ring/jobs/240986948 |
@pietro I had to revert these changes due to the build failures. Not sure what we can do. Maybe we can delete intermediate inputs after the input has been used and is no longer needed. And/or maybe when we need to rebuild a cached file, we should delete all the prevously- (stale) cached files before attempting the rebuild? |
I'm going to close this PR in favor of switching to |
This PR improves the Android situation on Travis CI. It caches some of the Android things we need to download for every build and adds AArch64 builds. It fixes both #486 and the issue I wrote about on #518.