-
Notifications
You must be signed in to change notification settings - Fork 169
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
Fix Travis CI build. #547
Fix Travis CI build. #547
Conversation
Lol, why u check CLA on forks? |
Though I am talking to myself in a lonely fork… who am i to judge a bot doing its work… |
Oh damn, it's not my fork, apologies to everyone! (I specifically clicked "New PR" on my own fork to test PR before opening it against upstream) |
Also bump the buck version to bring in the fix for facebook/buck#1510 since it was causing failures on travis. |
would be great if you can bump the buckversion too |
.travis.yml
Outdated
- echo $android_sdk_preview_license > $ANDROID_HOME/licenses/android-sdk-preview-license | ||
- export ANDROID_HOME="$HOME"/android-sdk | ||
- mkdir -p "$ANDROID_HOME" | ||
- export ANDROID_SDK_FILE_NAME=sdk-tools-darwin-3859397.zip |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why darwin? travis runs on linux right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah, facepalm, now I'm curious how it works in other projects where I did same thing… I guess Android tooling is more or less crossplatform now, that's how you check that lol
Will update buck version separately |
Ahh, I haven't have time to fix absence of |
Cmon travis, just trigger the build damn it :/