-
Notifications
You must be signed in to change notification settings - Fork 500
Use rerun-if
features to detect when code needs rebuilding
#230
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
Comments
I've historically avoided this for a few reasons:
The first point is a must-have for this feature and the latter is a "maybe this should be opt in" but probably not. |
Header dependency tracking is something of a solved problem AIUI - you just need to invoke the compiler with certain flags, so the real challenge there will be getting a portable solution. It looks easy enough with MSVC: https://docs.microsoft.com/en-gb/cpp/build/reference/showincludes-list-include-files And GCC has the WRT to the second item - aren't the lines output from the build script interpreted entirely independently by cargo? |
Yes my point is that this is not implemented, nor was it mentioned in the issue description. I wanted to outline why this isn't trivial to implement today and what needs to be done to close this issue. If you use |
Even if I manually add |
Yes, the output from the build file is saved to After a clean build there should be exactly two directories with your crate name: one of them will contain the binary of the compiled build script, and the other will contain the output file. |
So, am I correct in thinking that the I'm tracking down a rebuild issue, and I'm not sure if |
That is correct, |
In #443 (comment) you mention
Would this still be a good solution? I may be up for it when I have time, because I just got bit by missing a header. |
Issue rust-lang#230 is the issue for this, this patch just documents the current behaviour.
No description provided.
The text was updated successfully, but these errors were encountered: