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

fix rerun rule for build.rs #6

Merged
merged 1 commit into from
Nov 2, 2023
Merged

fix rerun rule for build.rs #6

merged 1 commit into from
Nov 2, 2023

Conversation

keiichiw
Copy link
Collaborator

@keiichiw keiichiw commented Nov 2, 2023

We had unneeded double-quotes for rerun-if-changed due to Debug formatting of Path, which caused rebuilding every time

We had unneeded double-quotes for rerun-if-changed due to Debug
formatting of Path, which caused rebuilding every time
@keiichiw keiichiw merged commit 49ef023 into main Nov 2, 2023
2 checks passed
@keiichiw keiichiw deleted the fix-build.rs branch November 2, 2023 13:50
crosvm-bot pushed a commit to google/crosvm that referenced this pull request Nov 6, 2023
With a fix in libtracemd-rs v0.2.1 [1], we no longer need to rebuild
libtracecmd-rs every time.
[1]: google/libtracecmd-rs#6

BUG=none
TEST=cargo build

Change-Id: Ica1342c9b23ed778fff2c0677717e3953b42dbd9
Reviewed-on: https://chromium-review.googlesource.com/c/crosvm/crosvm/+/5000825
Auto-Submit: Keiichi Watanabe <keiichiw@chromium.org>
Reviewed-by: Ryuichiro Chiba <chibar@chromium.org>
Reviewed-by: Morg <morg@google.com>
Commit-Queue: Keiichi Watanabe <keiichiw@chromium.org>
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

1 participant