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

Use modern language rules load statements #310

Merged
merged 3 commits into from
Dec 16, 2024

Conversation

martis42
Copy link
Owner

User facing changes:
Updating minimum required version for rules_cc to 0.15.0

@martis42 martis42 force-pushed the use_modern_language_rules_load_statements branch 2 times, most recently from 9e69657 to e310f5d Compare December 14, 2024 09:06
We upgrade to 0.15.0 due to:
- we intend to use load statements which are available starting with 0.10.0.
- versions 0.10.0 - 0.13.0 are not compatible to Bazel 6
- version 0.14.0 removed the still used `cc_proto_library` symbol
- version 0.15.0 restored `cc_proto_library` and is compatible to Bazel 6
Loading all public symbols from `<language>/defs.bzl` is a deprecated
approach for language rules. More fine grained loading prevents Bazel
analyzing things unrelated to what one actually is interested in.
This impacts us, as only for the fine grained loading corresponding
bzl_library targets exists we need for creating our documentation with
stardoc.
@martis42 martis42 force-pushed the use_modern_language_rules_load_statements branch from e310f5d to 6085d9a Compare December 16, 2024 13:09
Loading all public symbols from `<language>/defs.bzl` is a deprecated
approach for language rules. More fine grained loading prevents Bazel
analyzing things unrelated to what one actually is interested in.
We do this now to be consistent to how we use `rules_cc`.
@martis42 martis42 force-pushed the use_modern_language_rules_load_statements branch from 6085d9a to 70781e3 Compare December 16, 2024 13:11
@martis42 martis42 merged commit 5ab007a into main Dec 16, 2024
11 checks passed
@martis42 martis42 deleted the use_modern_language_rules_load_statements branch December 16, 2024 13:42
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.

1 participant