From c80575d26bb9fb4ef0c12ef08c6408e94e0c7c4c Mon Sep 17 00:00:00 2001 From: Richard Levasseur Date: Sun, 1 Feb 2026 12:10:46 -0800 Subject: [PATCH] chore: remove defunct `_py_toolchain_type` py_binary attribute --- python/private/py_executable.bzl | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/python/private/py_executable.bzl b/python/private/py_executable.bzl index 92fca2caf2..febb4a366e 100644 --- a/python/private/py_executable.bzl +++ b/python/private/py_executable.bzl @@ -62,7 +62,7 @@ load(":py_internal.bzl", "py_internal") load(":py_runtime_info.bzl", "DEFAULT_STUB_SHEBANG") load(":reexports.bzl", "BuiltinPyInfo", "BuiltinPyRuntimeInfo") load(":rule_builders.bzl", "ruleb") -load(":toolchain_types.bzl", "EXEC_TOOLS_TOOLCHAIN_TYPE", "TARGET_TOOLCHAIN_TYPE", TOOLCHAIN_TYPE = "TARGET_TOOLCHAIN_TYPE") +load(":toolchain_types.bzl", "EXEC_TOOLS_TOOLCHAIN_TYPE", TOOLCHAIN_TYPE = "TARGET_TOOLCHAIN_TYPE") load(":transition_labels.bzl", "TRANSITION_LABELS") load(":venv_runfiles.bzl", "create_venv_app_files") @@ -221,12 +221,6 @@ accepting arbitrary Python versions. # empty target for other platforms. default = "//tools/launcher:launcher", ), - # TODO: This appears to be vestigial. It's only added because - # GraphlessQueryTest.testLabelsOperator relies on it to test for - # query behavior of implicit dependencies. - "_py_toolchain_type": attr.label( - default = TARGET_TOOLCHAIN_TYPE, - ), "_python_version_flag": lambda: attrb.Label( default = labels.PYTHON_VERSION, ),