Skip to content

Overridable debuginfo type#945

Merged
tonyaiuto merged 2 commits intobazelbuild:mainfrom
nicbadiu:patch-3
Jul 31, 2025
Merged

Overridable debuginfo type#945
tonyaiuto merged 2 commits intobazelbuild:mainfrom
nicbadiu:patch-3

Conversation

@nicbadiu
Copy link
Contributor

@nicbadiu nicbadiu commented Mar 12, 2025

This change enables the override of the auto-detected /etc/os-release-based debuginfo_type attribute of the build_repo_for_rpmbuild_toolchain repository rule and allows the auto-detection by means of a new default option.

This also enables the use of none as an override option.

This change also includes a regeneration of the documentation, which seems to have trailed recent code changes. But there are no documentation changes related to this change as build_repo_for_rpmbuild_toolchain repository rule is not documented (yet).

@nicbadiu nicbadiu requested a review from nacl as a code owner March 12, 2025 09:16
@nicbadiu nicbadiu force-pushed the patch-3 branch 2 times, most recently from 15732c6 to 4ea728f Compare March 12, 2025 09:22
@nicbadiu nicbadiu requested review from aiuto and cgrindel as code owners March 12, 2025 09:22
@tonyaiuto
Copy link
Collaborator

@kellyma2 Do you want to take a look at it.

@nicbadiu Can you provide a description of the API change in the PR description?
Are you able to update the docs too? See the How To of doc_build/BUILD

@nicbadiu nicbadiu force-pushed the patch-3 branch 3 times, most recently from 440204a to 4904055 Compare July 29, 2025 15:39
This change enables the override of the auto-detected `/etc/os-release`-based
`debuginfo` type and allows the auto-detection by means of a new  `default`
option.

This also enables the use of `none` as an override option.
@nicbadiu
Copy link
Contributor Author

@kellyma2 Do you want to take a look at it.

@nicbadiu Can you provide a description of the API change in the PR description? Are you able to update the docs too? See the How To of doc_build/BUILD

The description should be more targeted now.

The docs are updated - which were behind other changes it seems.

@tonyaiuto
Copy link
Collaborator

phooey. windows tests failing for no apparent reason. Have to look later.

@tonyaiuto
Copy link
Collaborator

It looks like you did this

+        rctx.watch(RELEASE_PATH)
+        if rctx.path(RELEASE_PATH).exists:

instead of

+        if rctx.path(RELEASE_PATH).exists:
+           rctx.watch(RELEASE_PATH)

I patch the PR.

@tonyaiuto tonyaiuto merged commit 8c7c2cf into bazelbuild:main Jul 31, 2025
2 of 3 checks passed
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.

4 participants