Skip to content

Include both windows and unix files in embedded_zipper_sources#29057

Closed
yuzhy8701 wants to merge 1 commit intobazelbuild:masterfrom
yuzhy8701:zipper-cross-platform
Closed

Include both windows and unix files in embedded_zipper_sources#29057
yuzhy8701 wants to merge 1 commit intobazelbuild:masterfrom
yuzhy8701:zipper-cross-platform

Conversation

@yuzhy8701
Copy link
Copy Markdown
Contributor

@yuzhy8701 yuzhy8701 commented Mar 24, 2026

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

This filegroup is being built for user's exec platform, which can be different from bazel's platform (= user's host platform).

This fixes the "input file not found" error when building with a remote execution platform that is different from the host.
@github-actions github-actions Bot added the awaiting-review PR is awaiting review from an assigned reviewer label Mar 24, 2026
Copy link
Copy Markdown
Collaborator

@fmeum fmeum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yuzhy8701
Copy link
Copy Markdown
Contributor Author

@meteorcloudy could you help merge it? I hope to get it fixed with 9.0.2.

@meteorcloudy meteorcloudy added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels Mar 26, 2026
@iancha1992 iancha1992 added the team-Rules-CPP Issues for C++ rules label Mar 26, 2026
@github-actions github-actions Bot removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Mar 27, 2026
iancha1992 pushed a commit to iancha1992/bazel that referenced this pull request Apr 6, 2026
…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 pushed a commit to iancha1992/bazel that referenced this pull request Apr 6, 2026
…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
github-merge-queue Bot pushed a commit that referenced this pull request Apr 7, 2026
#29235)

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

---------

Co-authored-by: Zach Yu <[email protected]>
github-merge-queue Bot pushed a commit that referenced this pull request Apr 7, 2026
#29236)

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

---------

Co-authored-by: Zach Yu <[email protected]>
sync-agent-neo Bot pushed a commit to aosp-mirror-neo/platform_prebuilts_bazel_linux-x86_64 that referenced this pull request Apr 21, 2026
This binary is based on bazel 9.0.1, with the following patches on top:

* bazelbuild/bazel#28810
* bazelbuild/bazel#29057
* cl/890821032

It resolves build problems with bazel 9 when targeting windows. The patches will
be included in a future version of bazel, which we'll upgrade to when released.

Bug: 404623658
Change-Id: I4d0e56270c9fb6565eb5ccda6a135f486a6a6964
sync-agent-neo Bot pushed a commit to aosp-mirror-neo/platform_prebuilts_bazel_windows-x86_64 that referenced this pull request Apr 21, 2026
This binary is based on bazel 9.0.1, with the following patches on top:

* bazelbuild/bazel#28810
* bazelbuild/bazel#29057
* cl/890821032

It resolves build problems with bazel 9 when targeting windows. The patches will
be included in a future version of bazel, which we'll upgrade to when released.

Bug: 404623658
Change-Id: Idd6d5dde35d57b1654cefdebc4c9f2db6a6a6964
sync-agent-neo Bot pushed a commit to aosp-mirror-neo/platform_prebuilts_bazel_darwin-x86_64 that referenced this pull request Apr 21, 2026
This binary is based on bazel 9.0.1, with the following patches on top:

* bazelbuild/bazel#28810
* bazelbuild/bazel#29057
* cl/890821032

It resolves build problems with bazel 9 when targeting windows. The patches will
be included in a future version of bazel, which we'll upgrade to when released.

Bug: 404623658
Change-Id: Ibf2ebcde8ae7acb526fc466c689e55f96a6a6964
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.

4 participants