Skip to content

[Bazel] Add bzl_library targets#2709

Open
RyanDraves wants to merge 1 commit intoraspberrypi:developfrom
RyanDraves:develop
Open

[Bazel] Add bzl_library targets#2709
RyanDraves wants to merge 1 commit intoraspberrypi:developfrom
RyanDraves:develop

Conversation

@RyanDraves
Copy link
Copy Markdown

Add some bzl_library targets for .bzl files. This enables stardoc to do its thing and is helpful in its own right for validating rule & macro dependencies.

Notes:

  • Open question: should this PR add bzl_library targets for all .bzl files? I added the ones I personally depend on, but adding it for more files requires bumping the rules_cc version; might be poking the bear?
  • I haven't filed an issue yet, will do as desired

@lurch lurch added the build label Nov 30, 2025
@kilograham
Copy link
Copy Markdown
Contributor

cc @armandomontanez

@armandomontanez
Copy link
Copy Markdown
Contributor

Bumping rules_cc should be fine.

I should probably take a pass at some of the .bzl libraries and re-assess their intended visibility. For example, configurable_feature was an implementation detail to make it easier to write/maintain the toolchains. I don't think it should be considered part of the public interface.

Btw, thanks for kicking this off, stardoc in the BCR is very cool.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants