Skip to content

feat(cc): populate abi and platform settings on local py_cc_toolchain - #3990

Merged
rickeylev merged 3 commits into
bazel-contrib:mainfrom
rickeylev:update_py_cc_toolchain_runtime
Aug 4, 2026
Merged

feat(cc): populate abi and platform settings on local py_cc_toolchain#3990
rickeylev merged 3 commits into
bazel-contrib:mainfrom
rickeylev:update_py_cc_toolchain_runtime

Conversation

@rickeylev

Copy link
Copy Markdown
Collaborator

feat(cc): populate abi and platform settings on local py_cc_toolchain

The introduction of py_extension added abi_flags, abi_tag, platform_machine, and sys_platform to py_cc_toolchain. Local Python runtime toolchains did not populate these fields, causing py_extension builds to fail when using local toolchains.

Extract sys_platform, platform_machine, and abi_tag (from sysconfig SOABI) in get_local_runtime_info.py, pass them through local_runtime_repo.bzl, and set them on py_cc_toolchain in local_runtime_repo_setup.bzl.

The introduction of py_extension added abi_flags, abi_tag, platform_machine,
and sys_platform to py_cc_toolchain. Local python runtime toolchains did not
populate these fields, causing py_extension builds to fail when using local
toolchains.

Extract sys_platform, platform_machine, and abi_tag (from sysconfig SOABI) in
get_local_runtime_info.py, pass them through local_runtime_repo.bzl, and set
them on py_cc_toolchain in local_runtime_repo_setup.bzl.
@rickeylev
rickeylev requested a review from aignas as a code owner August 2, 2026 23:34
@rickeylev
rickeylev force-pushed the update_py_cc_toolchain_runtime branch from 551fa1b to 1d96f18 Compare August 3, 2026 00:11
@rickeylev
rickeylev enabled auto-merge August 3, 2026 05:38
@rickeylev
rickeylev added this pull request to the merge queue Aug 4, 2026
Merged via the queue into bazel-contrib:main with commit c73f931 Aug 4, 2026
6 checks passed
@rickeylev
rickeylev deleted the update_py_cc_toolchain_runtime branch August 4, 2026 04:59
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