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

ci: when installing rust-toolchain-installer-master, install it in debug mode to save some time on ci. #3389

Merged
merged 1 commit into from
Nov 2, 2018

Conversation

matthiaskrgr
Copy link
Member

the compiletime optimizations probably take longer than the speedup we get when executing the optimized binary vs debug build.

…bug mode to save some time in ci.

the compiletime optimizations probably take longer than the speedup we get when executing the optimized binary vs debug build.
@matthiaskrgr
Copy link
Member Author

Let's see if there are noticeable differences in wall time.

@matthiaskrgr
Copy link
Member Author

Total build time is reduced by ~1 hour.
Jobs see a couple of minutes speedup each.

master: https://travis-ci.org/rust-lang-nursery/rust-clippy/builds/448715468
PR: https://travis-ci.org/rust-lang-nursery/rust-clippy/builds/448773511

@flip1995
Copy link
Member

Effective time reduction is 17 mins, since the jobs run in parallel. Which is still an improvement!

@matthiaskrgr
Copy link
Member Author

When "normal" users (non-rust-lang-nursery repos) do travis builds I expect this change to have a more positive impact since by default users only have 4 or 5 parallel builds available on travis-ci.

@matthiaskrgr
Copy link
Member Author

The only danger I can think of right now is that we could hit some kind of debug assert or overflows that would normally be optimized away/not emitted in release mode while running the installer, but it doesn't looks like the root crate containes any assert!s or debug_assert!s...

@flip1995
Copy link
Member

We could just merge this and see how it plays out on the next few PRs. If it fails too often we can disable it again, no harm done.

@phansch
Copy link
Member

phansch commented Nov 1, 2018

LGTM, let's see how it works out 👍

bors r+

bors bot added a commit that referenced this pull request Nov 1, 2018
3389: ci: when installing rust-toolchain-installer-master, install it in debug mode to save some time on ci. r=phansch a=matthiaskrgr

the compiletime optimizations probably take longer than the speedup we get when executing the optimized binary vs debug build.

Co-authored-by: Matthias Krüger <matthias.krueger@famsik.de>
@bors
Copy link
Contributor

bors bot commented Nov 1, 2018

Build failed

@phansch
Copy link
Member

phansch commented Nov 1, 2018

Looks like nightly-x86_64-apple-darwin is missing in the latest nightly: https://mexus.github.io/rustup-components-history/x86_64-apple-darwin.html.
This makes the Mac OS-X job fail. I opened rust-lang/rust#55571.

@phansch
Copy link
Member

phansch commented Nov 2, 2018

bors r+

bors bot added a commit that referenced this pull request Nov 2, 2018
3389: ci: when installing rust-toolchain-installer-master, install it in debug mode to save some time on ci. r=phansch a=matthiaskrgr

the compiletime optimizations probably take longer than the speedup we get when executing the optimized binary vs debug build.

Co-authored-by: Matthias Krüger <matthias.krueger@famsik.de>
@bors
Copy link
Contributor

bors bot commented Nov 2, 2018

@bors bors bot merged commit 59f4aba into rust-lang:master Nov 2, 2018
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.

3 participants