-
Notifications
You must be signed in to change notification settings - Fork 446
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 Bzlmod #5066
Comments
Could this help resolve some diamond dependency problems we have? (e.g., p4runtime vs p4-constraints vs p4c dependency conflicts) Or fix #4640 where we can not export some feature flags? |
Yes, in the sense that it will make sure that exactly one version of each common dependency will be picked.
Not that I know off, no. |
The issue is effectively: abseil/abseil-cpp#740 We are working around it by setting some options in CMake, but it would be nice to be consistent. Abseil is the only third-party system dependency in the compiler that requires quote includes. |
This specifically enables the legacy WORKSPACE-based resolution, and disables the new bzlmod-based resolution. Related to p4lang#5065 and p4lang#5066.
This specifically enables the legacy WORKSPACE-based resolution, and disables the new bzlmod-based resolution. Related to p4lang#5065 and p4lang#5066. Signed-off-by: John Cater <[email protected]>
This specifically enables the legacy WORKSPACE-based resolution, and disables the new bzlmod-based resolution. Related to #5065 and #5066. --------- Signed-off-by: John Cater <[email protected]>
Bazel finally added a external dependency management system, called "Bzlmod", a while back: https://bazel.build/external/overview
By now, Bzlmod is mature and the old system (WORKSPACE.bazel files) have been deprecated as of Bazel 8 and will be removed in Bazel 9: https://bazel.build/external/migration
It is time to migrate to future proof our code base. This issue is tracking that effort.
There is a related issue that tracks upgrading to Bazel 8: #5065
I created the same issue for p4runtime and p4-constraints:
The text was updated successfully, but these errors were encountered: