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

Fixes _module_repo_name when building with Bazel@HEAD or Bazel 7.1 #21486

Conversation

fweikert
Copy link
Member

If get_canonical_repo_name no longer returns the repo name with version due to containing a54a393, the _module_repo_name should not either.

Fixes: #21292

Closes #21324.

PiperOrigin-RevId: 606646238
Change-Id: I8835a84842c2c66929586b39156eb9f5a541652f

If `get_canonical_repo_name` no longer returns the repo name with version due to containing bazelbuild@a54a393, the `_module_repo_name` should not either.

Fixes: bazelbuild#21292

Closes bazelbuild#21324.

PiperOrigin-RevId: 606646238
Change-Id: I8835a84842c2c66929586b39156eb9f5a541652f
@github-actions github-actions bot added the awaiting-review PR is awaiting review from an assigned reviewer label Feb 23, 2024
@fweikert
Copy link
Member Author

fweikert commented Feb 23, 2024

Cherrypick is needed to fix 8.0.0-pre.20240213.1: https://buildkite.com/bazel-trusted/bazel-release/builds/1359

#21473

@fweikert fweikert merged commit f4205cf into bazelbuild:release-8.0.0-pre.20240213.1rc1 Feb 23, 2024
33 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Feb 23, 2024
@fweikert fweikert deleted the release-8.0.0-pre.20240213.1rc1 branch February 23, 2024 15:12
fweikert added a commit to fweikert/bazel that referenced this pull request Mar 5, 2024
…azelbuild#21486)

If `get_canonical_repo_name` no longer returns the repo name with
version due to containing
bazelbuild@a54a393,
the `_module_repo_name` should not either.

Fixes: bazelbuild#21292

Closes bazelbuild#21324.

PiperOrigin-RevId: 606646238
Change-Id: I8835a84842c2c66929586b39156eb9f5a541652f

Co-authored-by: Yun Peng <pcloudy@google.com>
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.

3 participants