-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Mention sccache in the guide #4464
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
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
src/doc/guide.md
Outdated
Cargo shares build artifacts among all the packages of a single workspaces. | ||
If you want to share build results across different workspaces, you can use | ||
[sccache]. For the minimal setup, install sccache with `cargo install sccache` | ||
and add `export RUSTC_WRAPPER=sccache` to the shell init script. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
export RUSTC_WRAPPER=sccache
is horribly Unix specific, but I don't know a better way of saying: "Hey, you need to set up this env var to this value, and you must do it on every login, it's not a one-time thing".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps this could mention the environment variable and then say something like "for example if you're using a bash shell you can do..."
src/doc/guide.md
Outdated
## Build cache | ||
|
||
Cargo shares build artifacts among all the packages of a single workspaces. | ||
If you want to share build results across different workspaces, you can use |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps this could explicitly call out that this feature isn't built-in to cargo today but sccache, a third party tool, can be used to achieve similar results?
f86e5e3
to
13bb7bd
Compare
Updated! |
@bors: r+ |
📌 Commit 13bb7bd has been approved by |
Mention sccache in the guide Closes #4307
☀️ Test successful - status-appveyor, status-travis |
Closes #4307