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

Try to improve build times #1350

Merged
merged 1 commit into from
Jan 21, 2024
Merged

Try to improve build times #1350

merged 1 commit into from
Jan 21, 2024

Conversation

mre
Copy link
Member

@mre mre commented Jan 21, 2024

@mre
Copy link
Member Author

mre commented Jan 21, 2024

Results:

  • First run: 3m 29s
  • Second run: 1m 54s
  • Third run: 1m 42s

Pretty good! Cached builds haven't been below 2 minutes in a while and are typically hovering around 3m 30s, so I call this an improvement, although I did not run exhaustive tests.

@mre mre merged commit 3558031 into master Jan 21, 2024
7 checks passed
@mre mre deleted the build-times branch January 21, 2024 00:55
@MarcoIeni
Copy link

it seems that rust-cache automatically set CARGO_INCREMENTAL.
From the readme:
image

@mre
Copy link
Member Author

mre commented Jan 27, 2024

Oh, I did not know that. Thanks for sharing!
Couldn't harm to explicitly add it, just in case we move to a different caching solution in the future (even though I doubt that will happen anytime soon; rust-cache is pretty awesome).

@MarcoIeni
Copy link

yeah, I just wanted to share this :)
Great article btw :)
I didn't know about cargo-features-manager

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.

None yet

2 participants