Skip to content

[9.1.0] Include both windows and unix files in embedded_zipper_sources#29236

Merged
iancha1992 merged 2 commits intobazelbuild:release-9.1.0from
iancha1992:cp5cc93eef35-9.1.0
Apr 7, 2026
Merged

[9.1.0] Include both windows and unix files in embedded_zipper_sources#29236
iancha1992 merged 2 commits intobazelbuild:release-9.1.0from
iancha1992:cp5cc93eef35-9.1.0

Conversation

@iancha1992
Copy link
Copy Markdown
Member

Description

This filegroup is being built for user's exec platform, which can be different from bazel's platform (= user's host platform). The select() selects with bazel's platform, making it impossible to build bazel's internal tools for a different platform.

Motivation

This fixes the "input file not found" error when building with a remote execution platform that is different from the host.

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: None

Closes #29057.

PiperOrigin-RevId: 890318692
Change-Id: Ibfd0ae8d934e2e0ca9726a0ee907b4791c346504

Commit 919c0d4 and 5cc93ee

Guarding against mingw32 is not sufficient when a custom non-mingw32 toolchain injects the type. The added guard allows the toolchain to override this (and only this) typedef.

PiperOrigin-RevId: 892509663
Change-Id: I05e7420ccaa73f49c3038e6e67bf32f6661af120
…elbuild#29057)

### Description

This filegroup is being built for user's exec platform, which can be different from bazel's platform (= user's host platform). The `select()` selects with bazel's platform, making it impossible to build bazel's internal tools for a different platform.

### Motivation

This fixes the "input file not found" error when building with a remote execution platform that is different from the host.

### 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: None

Closes bazelbuild#29057.

PiperOrigin-RevId: 890318692
Change-Id: Ibfd0ae8d934e2e0ca9726a0ee907b4791c346504
@iancha1992 iancha1992 requested a review from a team as a code owner April 6, 2026 18:52
@iancha1992 iancha1992 added team-Rules-CPP Issues for C++ rules awaiting-review PR is awaiting review from an assigned reviewer labels Apr 6, 2026
@iancha1992 iancha1992 enabled auto-merge April 6, 2026 18:52
@iancha1992
Copy link
Copy Markdown
Member Author

For ref, please see #28967 (comment)

@iancha1992 iancha1992 added this to the 9.1.0 release blockers milestone Apr 6, 2026
@iancha1992 iancha1992 added this pull request to the merge queue Apr 7, 2026
Merged via the queue into bazelbuild:release-9.1.0 with commit f9dfdb9 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-Rules-CPP Issues for C++ rules

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants