Skip to content

fix(toolchains): include libpython in python_repository attrs - #4382

Merged
rickeylev merged 2 commits into
bazel-contrib:mainfrom
rickeylev:suppress_libpython_repository_warning
Oct 30, 2026
Merged

rickeylev merged 2 commits into
bazel-contrib:mainfrom
rickeylev:suppress_libpython_repository_warning

Conversation

@rickeylev

Copy link
Copy Markdown
Collaborator

_python_repository_impl omitted libpython from its returned
reproducibility attrs dict, causing Bazel < 8.3.0 and non-sha256
invocations to emit a spurious DEBUG warning about dropping
["libpython"].

Include libpython in attrs so Bazel preserves the attribute
without warning.

`_python_repository_impl` omitted `libpython` from its returned
reproducibility `attrs` dict, causing Bazel < 8.3.0 and non-`sha256`
invocations to emit a spurious `DEBUG` warning about dropping
`["libpython"]`.

Include `libpython` in `attrs` so Bazel preserves the attribute
without warning.
@rickeylev
rickeylev requested a review from aignas as a code owner October 30, 2026 01:19
@rickeylev

Copy link
Copy Markdown
Collaborator Author

AI Disclosure

  • Human oversight: semi-autonomous — this PR is human-directed and
    human-reviewed, with human sign-off on changes.
  • Comment handling: The human author reviews PR feedback and either
    responds directly or reviews and approves any agent-assisted updates
    before they are posted.

This is an automatically generated agent comment.

The debug message only appeared under repository debugging or older
Bazel versions and is too minor for release notes. Remove the news
fragment.
@rickeylev
rickeylev merged commit bfbcc01 into bazel-contrib:main Oct 30, 2026
13 checks passed
@rickeylev
rickeylev deleted the suppress_libpython_repository_warning branch October 30, 2026 07:28
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