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

Exclude .tmp_git_root from globs #1948

Merged
merged 2 commits into from
Jun 23, 2023

Conversation

cameron-martin
Copy link
Contributor

This directory contains the original git repo when the crate is from git. Including this directory currently makes these rules re-run whenever the repository rule re-runs, although this is fixed by bazelbuild/bazel#18271. Even after this fix, excluding this directory avoids depending on unnecessary files.

Fixes #1927.

@cameron-martin
Copy link
Contributor Author

The only actual change is in crate_universe/src/rendering.rs and the other changes are re-generated the vendored BUILD files.

Copy link
Collaborator

@UebelAndre UebelAndre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Can you separate this into two different commits? I also think if you retry the job it will succeed.

@cameron-martin
Copy link
Contributor Author

I've split this up, although not sure why the CI is failing. Looks like its trying to build windows targets on linux.

@UebelAndre
Copy link
Collaborator

@scentini @krasimirgg The build failures here doesn't make much sense. What is explicitly requesting the following target in this change?

(15:32:19) ERROR: Target //test/linker_inputs_propagation:depends_on_foo_via_sharedlib is incompatible and cannot be built, but was explicitly requested.

@UebelAndre UebelAndre requested a review from scentini May 3, 2023 15:37
@scentini
Copy link
Collaborator

scentini commented Jun 20, 2023

Please don't merge on green CI ( I triggered a rebuild ) as it's not really green, if you look at https://buildkite.com/bazel/rules-rust-rustlang/builds/8831#0188d87a-2695-4598-a8ad-3c8d9581f3e4 the task crashed.

@scentini
Copy link
Collaborator

I'm trying to figure out what's going on here.

@scentini
Copy link
Collaborator

Current CI run was correct in skipping the incompatible targets; seems like it's coverage that causes issues.

@scentini
Copy link
Collaborator

Looks like previous CI run was with Bazel 6.1.2 while my current is with 6.2.1. Release notes for 6.2.0 include changes related with target_compatible_with, so I'm fine assuming that this was fixed on the Bazel side.
Seems like @UebelAndre has already raised the issue wrt BEP trying to upload missing .dat files over at bazelbuild/bazel#17001. Let me know if the review is still blocked on me.

This directory contains the original git repo when the crate is from git. Including this directory currently makes these rules re-run whenever the repository rule re-runs, although this is fixed by bazelbuild/bazel#18271. Even after this fix, excluding this directory avoids depending on unnecessary files.

Fixes bazelbuild#1927.
@cameron-martin
Copy link
Contributor Author

@UebelAndre looks like CI has passed now, and I've regenerated the vendored files.

Copy link
Collaborator

@UebelAndre UebelAndre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for fixing this!

@UebelAndre UebelAndre merged commit 36f8251 into bazelbuild:main Jun 23, 2023
freeformstu pushed a commit to freeformstu/rules_rust that referenced this pull request Jun 24, 2023
* Exclude .tmp_git_root from globs

This directory contains the original git repo when the crate is from git. Including this directory currently makes these rules re-run whenever the repository rule re-runs, although this is fixed by bazelbuild/bazel#18271. Even after this fix, excluding this directory avoids depending on unnecessary files.

Fixes bazelbuild#1927.

* Regenerate vendored crates
Silcet pushed a commit to Silcet/rules_rust that referenced this pull request Jul 10, 2023
* Exclude .tmp_git_root from globs

This directory contains the original git repo when the crate is from git. Including this directory currently makes these rules re-run whenever the repository rule re-runs, although this is fixed by bazelbuild/bazel#18271. Even after this fix, excluding this directory avoids depending on unnecessary files.

Fixes bazelbuild#1927.

* Regenerate vendored crates
Silcet pushed a commit to Silcet/rules_rust that referenced this pull request Jul 10, 2023
* Exclude .tmp_git_root from globs

This directory contains the original git repo when the crate is from git. Including this directory currently makes these rules re-run whenever the repository rule re-runs, although this is fixed by bazelbuild/bazel#18271. Even after this fix, excluding this directory avoids depending on unnecessary files.

Fixes bazelbuild#1927.

* Regenerate vendored crates
@cameron-martin cameron-martin deleted the exclude-git branch February 1, 2024 23:50
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.

Cargo crates generated from git repositories are not reproducible
3 participants