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

Update bzlmod support to use bazel_versions.bzl and reduce boilerplate #125

Closed
Tracked by #120
cgrindel opened this issue Mar 14, 2023 · 1 comment · Fixed by #132
Closed
Tracked by #120

Update bzlmod support to use bazel_versions.bzl and reduce boilerplate #125

cgrindel opened this issue Mar 14, 2023 · 1 comment · Fixed by #132
Labels
chore Code cleanup or refactor

Comments

@cgrindel
Copy link
Member

cgrindel commented Mar 14, 2023

Context and Ideas

Related to #120.

@cgrindel cgrindel added the chore Code cleanup or refactor label Mar 14, 2023
cgrindel added a commit that referenced this issue Mar 16, 2023
- Add `.bcr` folder with requisite files.
- Generate hub repository to aid in the resolution of the Bazel binary
repositories.
- Add `workspace_bazel_binaries` rule to allow repositories that use
`rules_bazel_integration_test` to work with and without bzlmod enabled.

Inspired by bazelbuild/bazel-gazelle#1423.
Related to #125.
@cgrindel
Copy link
Member Author

Opted to have bazel_binaries provide a versions struct that provides current, other, and all.

cgrindel added a commit that referenced this issue Mar 16, 2023
Clients that are using bzlmod now only need to specify the Bazel
versions in their `MODULE.bazel`. The `bazel_versions.bzl` is no longer
necessary.
- Update `bazel_binaries` to provide the version information.
- Update `workspace_bazel_binaries` to consume the version info so that
it can also provide it.

Closes #125.
cgrindel added a commit to k1nkreet/rules_bazel_integration_test that referenced this issue Sep 27, 2023
…rib#128)

- Add `.bcr` folder with requisite files.
- Generate hub repository to aid in the resolution of the Bazel binary
repositories.
- Add `workspace_bazel_binaries` rule to allow repositories that use
`rules_bazel_integration_test` to work with and without bzlmod enabled.

Inspired by bazelbuild/bazel-gazelle#1423.
Related to bazel-contrib#125.
cgrindel added a commit to k1nkreet/rules_bazel_integration_test that referenced this issue Sep 27, 2023
…rib#132)

Clients that are using bzlmod now only need to specify the Bazel
versions in their `MODULE.bazel`. The `bazel_versions.bzl` is no longer
necessary.
- Update `bazel_binaries` to provide the version information.
- Update `workspace_bazel_binaries` to consume the version info so that
it can also provide it.

Closes bazel-contrib#125.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Code cleanup or refactor
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant