/rust

travis: Don't use -9 on gzip #38609

Merged
merged 1 commit into from Dec 30, 2016

Reviewers

No reviews

Assignees

@japaric japaric

Labels

None yet

Projects

None yet

Milestone

No milestone

5 participants

@alexcrichton @rust-highfive @japaric @bors @nikomatsakis
@alexcrichton
Member
alexcrichton commented Dec 26, 2016

I timed this locally and plain old gzip took 2m06s while gzip -9 took a
whopping 6m23s to save a mere 4MB out of 1.2GB. Let's shave a few minutes off
the Android builder by turning down the compression level.
@alexcrichton alexcrichton travis: Don't use -9 on gzip 
I timed this locally and plain old `gzip` took 2m06s while `gzip -9` took a
whopping 6m23s to save a mere 4MB out of 1.2GB. Let's shave a few minutes off
the Android builder by turning down the compression level.
67cc77c
@rust-highfive
Collaborator
rust-highfive commented Dec 26, 2016

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)
@japaric
Member
japaric commented Dec 26, 2016

@bors r+
@bors
Contributor
bors commented Dec 26, 2016

📌 Commit 67cc77c has been approved by japaric
@japaric japaric assigned japaric and unassigned nikomatsakis Dec 26, 2016
@alexcrichton
Member
alexcrichton commented Dec 27, 2016

@bors: rollup
@alexcrichton alexcrichton added a commit to alexcrichton/rust that referenced this pull request Dec 30, 2016
@alexcrichton alexcrichton Rollup merge of #38609 - alexcrichton:less-compress, r=japaric 
travis: Don't use -9 on gzip

I timed this locally and plain old `gzip` took 2m06s while `gzip -9` took a
whopping 6m23s to save a mere 4MB out of 1.2GB. Let's shave a few minutes off
the Android builder by turning down the compression level.
 a68f886
@alexcrichton alexcrichton referenced this pull request Dec 30, 2016
Merged

Rollup of 25 pull requests #38697

@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 67cc77c into rust-lang:master Dec 30, 2016
1 check passed

1 check passed

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