Skip to content

[9.0.2] Fix launcher/launcher_maker for cross build (https://github.com/bazelbuild/bazel/pull/29200)#29238

Merged
iancha1992 merged 1 commit intobazelbuild:release-9.0.2from
bazel-io:cp29200-9.0.2-091153
Apr 7, 2026
Merged

[9.0.2] Fix launcher/launcher_maker for cross build (https://github.com/bazelbuild/bazel/pull/29200)#29238
iancha1992 merged 1 commit intobazelbuild:release-9.0.2from
bazel-io:cp29200-9.0.2-091153

Conversation

@bazel-io
Copy link
Copy Markdown
Member

@bazel-io bazel-io commented Apr 7, 2026

Description

Fix launcher/launcher_maker for cross build. launcher will be used in the build output, so it should use cfg=target. launcher_maker is used to combine a binary for target and launcher for target into the final output at build time, so it should use cfg=exec.

Motivation

To make py_binary launcher work when building on linux (exec) for windows (target).

Build API Changes

No

Checklist

  • I have added tests for the new use cases (if any).
  • I have updated the documentation (if applicable).

Release Notes

RELNOTES: Fix launcher/launcher_maker for cross build, e.g. build windows binary on linux machine.

Closes #29200.

PiperOrigin-RevId: 895761405
Change-Id: I0b3dd3431d25fb4fb9ea827f344eb5cf42c11276

Commit 4ff3127

### Description

Fix launcher/launcher_maker for cross build. launcher will be used in the build output, so it should use cfg=target. launcher_maker is used to combine a binary for target and launcher for target into the final output at build time, so it should use cfg=exec.

### Motivation

To make py_binary launcher work when building on linux (exec) for windows (target).

### Build API Changes

No

### Checklist

- [ ] I have added tests for the new use cases (if any).
- [ ] I have updated the documentation (if applicable).

### Release Notes

RELNOTES: Fix launcher/launcher_maker for cross build, e.g. build windows binary on linux machine.

Closes bazelbuild#29200.

PiperOrigin-RevId: 895761405
Change-Id: I0b3dd3431d25fb4fb9ea827f344eb5cf42c11276
@bazel-io bazel-io requested a review from a team as a code owner April 7, 2026 09:12
@bazel-io bazel-io added team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file. awaiting-review PR is awaiting review from an assigned reviewer labels Apr 7, 2026
@bazel-io bazel-io requested review from Wyverald and fmeum April 7, 2026 09:12
@iancha1992 iancha1992 requested review from meteorcloudy and removed request for fmeum April 7, 2026 15:33
@iancha1992 iancha1992 enabled auto-merge April 7, 2026 15:33
@iancha1992 iancha1992 added this pull request to the merge queue Apr 7, 2026
Merged via the queue into bazelbuild:release-9.0.2 with commit 8ac324e Apr 7, 2026
46 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants