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

feat(toolchain): add patch_strip attr for python_repository #2201

Merged
merged 6 commits into from
Sep 9, 2024

Conversation

aignas
Copy link
Collaborator

@aignas aignas commented Sep 7, 2024

This value has been hardcoded for a long time, so let's add the patch_strip
attribute to the WORKSPACE setups now so that we add less tech debt in the
bzlmod world later and need to migrate bzlmod users from the hard coded
value to something that can be configured.

This should be backwards compatible because of the default int value of 1
for the patch_strip attribute.

Summary:

  • feat: add patch_strip to python_repository.
  • feat: add the default value of patch_strip to get_release_info.
  • refactor: handle patch_strip key in TOOL_VERSIONS.

Work towards #2081.

python/private/python_repositories.bzl Outdated Show resolved Hide resolved
@rickeylev rickeylev added this pull request to the merge queue Sep 9, 2024
Merged via the queue into bazelbuild:main with commit c804a13 Sep 9, 2024
4 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.

2 participants