Skip to content

Add Windows MSVC Bazel targets for native voice libraries - #43144

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/fb2b4998e9ffb5b64ca830ebf4bb4633fc959eb9
Sep 6, 2026
Merged

copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/fb2b4998e9ffb5b64ca830ebf4bb4633fc959eb9

Conversation

@copyberry

@copyberry copyberry Bot commented Sep 6, 2026 •

Copy link
Copy Markdown

Add Windows MSVC Bazel targets for native voice libraries

What changed

  • Add explicit x64 and ARM64 targets for native builds, runtime preparation, and linking using the existing voice recipes. Each target requires native Windows execution of the matching architecture.
  • Declare compiler, SDK, Python, and CMake inputs; require an explicitly supplied Cygwin/pkgconf tool tree and a fixed SystemRoot. Validate installed tool selections against the manifest and declared files.
  • Pair Windows DLLs with SDK import libraries, preserve plugin and receipt runfiles, and omit Unix runtime-search flags. Use Python for portable payload copying.
  • Correct the MSVC ARM64 tool directory casing to HostArm64 and document provisioning and build commands.

The generic Rust-consumer aliases remain separate; native Windows Bazel execution and consumer validation are still needed to establish complete Windows voice support.

Testing

Add eight unit tests covering tool selection, path anchoring, invalid inputs, and DLL/import-library copying, plus x64 and ARM64 link smoke targets that reference gst_version.

## What changed

- Add explicit x64 and ARM64 targets for native builds, runtime preparation, and linking using the existing voice recipes. Each target requires native Windows execution of the matching architecture.
- Declare compiler, SDK, Python, and CMake inputs; require an explicitly supplied Cygwin/pkgconf tool tree and a fixed `SystemRoot`. Validate installed tool selections against the manifest and declared files.
- Pair Windows DLLs with SDK import libraries, preserve plugin and receipt runfiles, and omit Unix runtime-search flags. Use Python for portable payload copying.
- Correct the MSVC ARM64 tool directory casing to `HostArm64` and document provisioning and build commands.

The generic Rust-consumer aliases remain separate; native Windows Bazel execution and consumer validation are still needed to establish complete Windows voice support.

## Testing

Add eight unit tests covering tool selection, path anchoring, invalid inputs, and DLL/import-library copying, plus x64 and ARM64 link smoke targets that reference `gst_version`.

GitOrigin-RevId: fb2b4998e9ffb5b64ca830ebf4bb4633fc959eb9
@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/fb2b4998e9ffb5b64ca830ebf4bb4633fc959eb9 branch from adb6df9 to 1c40ffe Compare September 6, 2026 02:15
@copyberry
copyberry Bot merged commit 1c40ffe into main Sep 6, 2026
36 of 43 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/fb2b4998e9ffb5b64ca830ebf4bb4633fc959eb9 branch September 6, 2026 02:16
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 6, 2026

This branch was previously deployed

1 inactive deployment
issue-triage — 1c40ffe4 Deployed Sep 6, 2026 by wych42 via Generate label suggestions #42681
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants