Skip to content
This repository was archived by the owner on Dec 16, 2024. It is now read-only.

Comments

Re-enable Android states on Travis#145

Merged
bors-servo merged 3 commits intoservo:masterfrom
aneeshusa:enable-android-on-travis
Nov 3, 2015
Merged

Re-enable Android states on Travis#145
bors-servo merged 3 commits intoservo:masterfrom
aneeshusa:enable-android-on-travis

Conversation

@aneeshusa
Copy link
Contributor

The larger memory limits on the Trusty beta will
hopefully make these less flaky.

Review on Reviewable

The larger memory limits on the Trusty beta will
hopefully make these less flaky.
@Manishearth
Copy link
Member

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit f4a3e8a has been approved by Manishearth

@bors-servo
Copy link
Contributor

⌛ Testing commit f4a3e8a with merge b69e52e...

bors-servo pushed a commit that referenced this pull request Nov 1, 2015
Re-enable Android states on Travis

The larger memory limits on the Trusty beta will
hopefully make these less flaky.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/145)
<!-- Reviewable:end -->
@aneeshusa
Copy link
Contributor Author

I'd recommend trying this build a few times before merging to make sure it's not flaky - I know there have been problems in the past.

@Manishearth
Copy link
Member

@bors-servo r- force

@Manishearth Manishearth closed this Nov 1, 2015
@Manishearth Manishearth reopened this Nov 1, 2015
@Manishearth
Copy link
Member

@bors-servo r-

@Manishearth
Copy link
Member

@bors-servo force

@Manishearth Manishearth closed this Nov 1, 2015
@Manishearth
Copy link
Member

Keeping it closed till the travis build passes.

@aneeshusa
Copy link
Contributor Author

Ok, thanks. Looks like some of the package names may have changed, I'll take a look at it now.

@Manishearth
Copy link
Member

Cool

@Manishearth Manishearth reopened this Nov 1, 2015
@Manishearth Manishearth closed this Nov 1, 2015
@Manishearth Manishearth reopened this Nov 1, 2015
Some of the dependencies for Android require 32-bit versions
of certain package, so add the architecture and update the package
database (via refresh: True) before installing them.
@aneeshusa
Copy link
Contributor Author

I fixed the package installation problem, but it looks like it's running into disk space limits.

@aneeshusa
Copy link
Contributor Author

Do you know which components/versions of the SDK/NDK we need? It looks like it may be possible to selectively download the pieces we need: https://sites.google.com/a/android.com/tools/recent/updatingsdkfromcommand-line

@Manishearth
Copy link
Member

Not sure. @larsbergstrom ?

@larsbergstrom
Copy link
Contributor

Platform tools and API 18 at least (it's hard to tell what it actually installed). I suspect this will take a little bit of trial and error.

The NDK comes as a big blob. you just have to grab & unpack the binary then also install the specific toolkit (use cmdline in https://github.com/servo/servo/wiki/Building-for-Android#set-up-toolchain).

Use --filter to only download and extract necessary SDK components. This
is faster, uses less resources and gets around Travis disk space limits.
Update to latest SDK and NDK versions and use sha512 instead of sha1.
Use archive.extracted for better semantics and concision.
@aneeshusa
Copy link
Contributor Author

Looks like I got it to build cleanly on Travis! Shame about the build time though.

@Manishearth
Copy link
Member

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit b80f1bd has been approved by Manishearth

@bors-servo
Copy link
Contributor

⌛ Testing commit b80f1bd with merge 062a51e...

bors-servo pushed a commit that referenced this pull request Nov 3, 2015
Re-enable Android states on Travis

The larger memory limits on the Trusty beta will
hopefully make these less flaky.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/145)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

☀️ Test successful - travis

@bors-servo bors-servo merged commit b80f1bd into servo:master Nov 3, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants