Skip to content

Conversation

@gregestren
Copy link
Contributor

@gregestren gregestren commented Dec 3, 2025

This removes hard-coded flags --python_path, --experimental_python_import_all_repositories, and --incompatible_remove_ctx_bazel_py_fragment.

The first two are redefined in rules_python 1.7.0+ and linked into Bazel in #27792. This PR graveyards them but we can delete them outright when #27793 merges.

--incompatible_remove_ctx_bazel_py_fragment was added in Bazel 9 to allow toggling between the native and Starlark definitions.

By removing --incompatible_remove_ctx_bazel_py_fragment, this change removes the ability to revert to native definitions for post-9 Bazel.

For #26521 and bazel-contrib/rules_python#3252.

@gregestren gregestren changed the title DRAFT: test removing native Python flags. DRAFT: Delete BazelPythonConfiguration Dec 3, 2025
@gregestren gregestren marked this pull request as ready for review December 3, 2025 23:21
@gregestren gregestren requested a review from rickeylev as a code owner December 3, 2025 23:21
@gregestren gregestren requested a review from dabanki December 3, 2025 23:22
@github-actions github-actions bot added team-Rules-Python Native rules for Python awaiting-review PR is awaiting review from an assigned reviewer labels Dec 3, 2025
@gregestren gregestren added the team-Configurability platforms, toolchains, cquery, select(), config transitions label Dec 3, 2025
@gregestren gregestren changed the title DRAFT: Delete BazelPythonConfiguration Delete BazelPythonConfiguration Dec 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-review PR is awaiting review from an assigned reviewer team-Configurability platforms, toolchains, cquery, select(), config transitions team-Rules-Python Native rules for Python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants