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

rustbuild: Add more deps on android-copy-libs #38729

Merged
merged 1 commit into from Dec 31, 2016

Conversation

alexcrichton
Copy link
Member

The android-copy-libs step is crucial for running tests on the Android target as
it copies necessary scripts and such to the emulator. We must run that before
running any tests there, but we erroneously only did it for compiletest test
suites!

The android-copy-libs step is crucial for running tests on the Android target as
it copies necessary scripts and such to the emulator. We must run that before
running any tests there, but we erroneously only did it for compiletest test
suites!
@rust-highfive
Copy link
Collaborator

r? @aturon

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton
Copy link
Member Author

I believe this will fix this failure: #38709 (comment)

@brson
Copy link
Contributor

brson commented Dec 31, 2016

@bors r+ p=1

@bors
Copy link
Contributor

bors commented Dec 31, 2016

📌 Commit cf8fde1 has been approved by brson

@bors
Copy link
Contributor

bors commented Dec 31, 2016

⌛ Testing commit cf8fde1 with merge 8c547a0...

bors added a commit that referenced this pull request Dec 31, 2016
rustbuild: Add more deps on android-copy-libs

The android-copy-libs step is crucial for running tests on the Android target as
it copies necessary scripts and such to the emulator. We must run that before
running any tests there, but we erroneously only did it for compiletest test
suites!
@bors
Copy link
Contributor

bors commented Dec 31, 2016

☀️ Test successful - status-appveyor, status-travis
Approved by: brson
Pushing 8c547a0 to master...

@bors bors merged commit cf8fde1 into rust-lang:master Dec 31, 2016
@alexcrichton alexcrichton deleted the android-steps branch January 1, 2017 00:24
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

5 participants