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

No cache at all! #72

Merged
merged 1 commit into from
Mar 3, 2019
Merged

No cache at all! #72

merged 1 commit into from
Mar 3, 2019

Conversation

jamesmunns
Copy link
Member

As opposed to #71, this disables caching completely. This is meant as a baseline comparison.

@jamesmunns
Copy link
Member Author

So, without caching, a build takes about 5 minutes for each of stable/beta/nightly, meaning a total of 15 minutes of build time.

@jamesmunns jamesmunns marked this pull request as ready for review March 3, 2019 22:12
@jamesmunns
Copy link
Member Author

It takes less time to download fresh and build than it does to use the cache. This PR disables caching, reducing the total build time from 25-30 minutes down to 15 or so.

@Yatekii Yatekii merged commit 1b25da4 into master Mar 3, 2019
@hannobraun hannobraun deleted the no-cache branch March 4, 2019 10:24
@hannobraun
Copy link
Contributor

@jamesmunns

It takes less time to download fresh and build than it does to use the cache. This PR disables caching, reducing the total build time from 25-30 minutes down to 15 or so.

Thanks for confirming. This matches my intuition, but I never bothered to actually measure it :-)

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