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

Allow additional targets to compile with bzlmod #2314

Merged
merged 4 commits into from
Dec 12, 2023

Conversation

matts1
Copy link
Contributor

@matts1 matts1 commented Dec 10, 2023

See #2246 for how we declare dependencies automatically for third party crates. I'm using the same mechanism for all other repositories.

Note that the MODULE.bazel changes in use_repo are autogenerated, and if you change the third party crates you depend on, you should get an error message saying something like "run this command to fix it" on your bzlmod presubmit environment.

@matts1 matts1 force-pushed the use_repo branch 2 times, most recently from b32196f to 54bd04a Compare December 10, 2023 23:50
Change-Id: I5d843f6b660731a35c6c4205b7d8c91b46716815
@matts1 matts1 force-pushed the use_repo branch 2 times, most recently from 46e291d to e786844 Compare December 11, 2023 00:03
rust/private/extensions.bzl Outdated Show resolved Hide resolved
MODULE.bazel Show resolved Hide resolved
@matts1
Copy link
Contributor Author

matts1 commented Dec 12, 2023

Build appears to be failing because of the upgrade to bazel 7.0.0. I created an empty PR that's failing these same presubmits (#2316)

@matts1
Copy link
Contributor Author

matts1 commented Dec 12, 2023

Filed #2317

@illicitonion illicitonion merged commit 44ffccb into bazelbuild:main Dec 12, 2023
3 checks passed
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.

2 participants