Sweep the cache before uploading it#246
Merged
yoshuawuyts merged 1 commit intohttp-rs:masterfrom May 20, 2019
Merged
Conversation
Contributor
Author
|
I wish there were an easy way to look at the cache content, the ones from this branch are still half a gig larger than my testing on travis-ci.com and the test job is taking twice as long. But at least comparing https://travis-ci.org/rustasync/tide/builds/535025048 and https://travis-ci.org/rustasync/tide/builds/535036746 the latter used about half the time for the test job. Once this is merged I might hit the "delete all caches" button on Travis and see if it creates a smaller cache inline with what I was seeing in the future. |
Nemo157
commented
May 20, 2019
| - cargo build | ||
| -Zmtime-on-use | ||
| --manifest-path tide/Cargo.toml | ||
| --no-default-features |
Contributor
Author
There was a problem hiding this comment.
rust-lang/cargo#6972 means that this does some extra work, but it's still faster than the test build so doesn't really matter at this point.
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Use
cargo-sweepalong with-Zmtime-on-usefor removing unused data from the Travis cache.cargo-sweepwill remove all files that were not used during the current build, so for example old versions of dependencies. To be able to track this accurately it needscargo/rustcto mark the files it uses during the current build, the-Zmtime-on-useflag is an unstable flag to tell it to do so.With this implemented the files uploaded to Travis should only be the relevant ones, keeping the cache minimal and actually usable. This will then mean that future builds will only have to build what has changed and run much faster.
Motivation and Context
Fixes #243
How Has This Been Tested?
Types of changes
Checklist: