-
Notifications
You must be signed in to change notification settings - Fork 277
Add support for sccache #1475
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
base: master
Are you sure you want to change the base?
Add support for sccache #1475
Conversation
|
Specifically, it depends on projg2/shadowman#4 |
|
I'm thinking about how rust could be integrated: should we export |
4f1042d to
5190029
Compare
|
Can you explain what this is useful for, given the build cache issue (it keys on the full path)? Are you really using it with live ebuilds enough for it to pay off? |
I've not thought about this at all. However I have flang live installed, so yes I could benefit from that. Also I'd be interested to support rust packages, where I also have a couple of live packages installed. Not that it would matter a lot. But I can see why this might not be worth the hassle. |
|
I would personally be less concerned about "how often is it even used, and is it worth the hassle" and be more concerned about the exact opposite problem: People will use it. And it will be a bad thing that they use it. They will enable it because they think it sounds like a good thing and then they will suffer a pure loss due to the cache fullpath issue. I'm strongly against merging sccache support in any form, unless it at least answers the question of how to make sure users have a fighting chance of understanding when it's a good idea vs when it's a bad idea. Mostly indifferent as long as that is handled. One possibility of the top of my head is to make the FEATURES=sccache be ignored unless PROPERTIES=live is also set? |
|
Isn't the path issue specific to Rust? |
|
No that happens for all supported compilers |
Might be worth the rename the feature flag as well to avoid confusion from the start |
Signed-off-by: idealseal <[email protected]>
This depends on projg2/shadowman#4 and a patched
sccacheebuild.It works exactly as the
ccacheimplementation.Docs are still missing. Maybe this can be combined with the
ccacheinstructions?