Skip to content

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

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Dec 30, 2016
Merged

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

merged 1 commit into from
Dec 30, 2016

Conversation

@alexcrichton
Copy link
Member

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
travis: Don't use -9 on gzip
67cc77c 
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.
@rust-highfive
Copy link
Contributor

r? @nikomatsakis

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

@japaric
Copy link
Contributor

japaric commented Dec 26, 2016

@bors r+

@bors
Copy link
Collaborator

bors commented Dec 26, 2016

📌 Commit 67cc77c has been approved by japaric

@japaric japaric assigned japaric and unassigned nikomatsakis Dec 26, 2016
@alexcrichton
Copy link
Member Author

@bors: rollup

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Dec 30, 2016
@alexcrichton
Rollup merge of rust-lang#38609 - alexcrichton:less-compress, r=japaric
a68f886 
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.
@alexcrichton alexcrichton mentioned this pull request Dec 30, 2016
Merged
bors added a commit that referenced this pull request Dec 30, 2016
@bors
Auto merge of #38697 - alexcrichton:rollup, r=alexcrichton
7f2d2af 
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
@bors bors merged commit 67cc77c into rust-lang:master Dec 30, 2016
@alexcrichton alexcrichton deleted the less-compress branch December 30, 2016 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@japaric japaric

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@alexcrichton @rust-highfive @japaric @bors @nikomatsakis