compiletest: Don't limit all suites on Android #38647

Merged
merged 1 commit into from Dec 30, 2016

Projects

None yet

4 participants

@alexcrichton
Member

On Android we only have one test thread for supposed problems with concurrency
and the remote debugger. Not all of our suites require one concurrency, however,
and suites like compile-fail or pretty can be much faster if they're
parallelized on Travis.

This commit only sets the test threads to one on Android for suites which
actually run code, and other suites aren't tampered with.

@alexcrichton alexcrichton compiletest: Don't limit all suites on Android
On Android we only have one test thread for supposed problems with concurrency
and the remote debugger. Not all of our suites require one concurrency, however,
and suites like compile-fail or pretty can be much faster if they're
parallelized on Travis.

This commit only sets the test threads to one on Android for suites which
actually run code, and other suites aren't tampered with.
f26e52e
@brson brson was assigned by rust-highfive Dec 28, 2016
@rust-highfive
Collaborator

r? @brson

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

@brson
Contributor
brson commented Dec 29, 2016

@bors r+

@bors
Contributor
bors commented Dec 29, 2016

📌 Commit f26e52e has been approved by brson

@alexcrichton alexcrichton added a commit to alexcrichton/rust that referenced this pull request Dec 30, 2016
@alexcrichton alexcrichton Rollup merge of #38647 - alexcrichton:faster-android, r=brson
compiletest: Don't limit all suites on Android

On Android we only have one test thread for supposed problems with concurrency
and the remote debugger. Not all of our suites require one concurrency, however,
and suites like compile-fail or pretty can be much faster if they're
parallelized on Travis.

This commit only sets the test threads to one on Android for suites which
actually run code, and other suites aren't tampered with.
332a4cc
@bors bors added a commit that referenced this pull request Dec 30, 2016
@bors bors Auto merge of #38697 - alexcrichton:rollup, r=alexcrichton
Rollup of 25 pull requests

- Successful merges: #37149, #38491, #38517, #38559, #38587, #38609, #38611, #38622, #38628, #38630, #38631, #38632, #38635, #38647, #38649, #38655, #38659, #38660, #38662, #38665, #38671, #38674, #38676, #38693, #38695
- Failed merges: #38657, #38680
4a68076
@bors bors added a commit that referenced this pull request Dec 30, 2016
@bors bors Auto merge of #38697 - alexcrichton:rollup, r=alexcrichton
Rollup of 25 pull requests

- Successful merges: #37149, #38491, #38517, #38559, #38587, #38609, #38611, #38622, #38628, #38630, #38631, #38632, #38635, #38647, #38649, #38655, #38659, #38660, #38662, #38665, #38671, #38674, #38676, #38693, #38695
- Failed merges: #38657, #38680
4536280
@bors bors added a commit that referenced this pull request Dec 30, 2016
@bors bors Auto merge of #38697 - alexcrichton:rollup, r=alexcrichton
Rollup of 25 pull requests

- Successful merges: #37149, #38491, #38517, #38559, #38587, #38609, #38611, #38622, #38628, #38630, #38631, #38632, #38635, #38647, #38649, #38655, #38659, #38660, #38662, #38665, #38671, #38674, #38676, #38693, #38695
- Failed merges: #38657, #38680
c4908ce
@bors bors added a commit that referenced this pull request Dec 30, 2016
@bors bors Auto merge of #38697 - alexcrichton:rollup, r=alexcrichton
Rollup of 25 pull requests

- Successful merges: #37149, #38491, #38517, #38559, #38587, #38609, #38611, #38622, #38628, #38630, #38631, #38632, #38635, #38647, #38649, #38655, #38659, #38660, #38662, #38665, #38671, #38674, #38676, #38693, #38695
- Failed merges: #38657, #38680
7f2d2af
@bors bors merged commit f26e52e into rust-lang:master Dec 30, 2016

1 check passed

Details continuous-integration/travis-ci/pr The Travis CI build passed
@alexcrichton alexcrichton deleted the alexcrichton:faster-android branch Dec 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment