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

Upgrade to Bazel 6.0 #18508

Closed
jwnimmer-tri opened this issue Jan 3, 2023 · 4 comments · Fixed by #18545
Closed

Upgrade to Bazel 6.0 #18508

jwnimmer-tri opened this issue Jan 3, 2023 · 4 comments · Fixed by #18545
Assignees
Labels
component: build system Bazel, CMake, dependencies, memory checkers, linters priority: medium type: feature request

Comments

@jwnimmer-tri
Copy link
Collaborator

We should upgrade our supported Bazel version to https://github.com/bazelbuild/bazel/releases/tag/6.0.0.

In #18246, I tested a couple of early release candidates and fixed some compatibility problems, so hopefully this is as simple as switching the setup scripts (.bazeliskrc and setup/ubuntu/source_distribution) plus the documentation.

@BetsyMcPhail
Copy link
Contributor

For reference, #16447 upgraded to 5.0

@jwnimmer-tri
Copy link
Collaborator Author

FYI one new thing compared to the 5.0 upgrade process --

On macOS today, our bazel version comes from a dotfile, which means that CI testing of the PR can use the Provisioned images (without any CI maintainer involvement).

The Ubuntu is unchanged -- it comes from a setup script, which means that CI testing of the PR needs to use the Unprovisioned job names.

@rpoyner-tri
Copy link
Contributor

Predictably, kcov integration needs love. #18615

@BetsyMcPhail
Copy link
Contributor

Linux provisioned images have been updated in CI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: build system Bazel, CMake, dependencies, memory checkers, linters priority: medium type: feature request
Development

Successfully merging a pull request may close this issue.

5 participants