Skip to content

[9.1.0] Fix Unicode encoding of uploaded directories (https://github.com/bazelbuild/bazel/pull/28684)#28703

Merged
iancha1992 merged 1 commit intobazelbuild:release-9.1.0from
bazel-io:cp28684-9.1.0
Feb 26, 2026
Merged

[9.1.0] Fix Unicode encoding of uploaded directories (https://github.com/bazelbuild/bazel/pull/28684)#28703
iancha1992 merged 1 commit intobazelbuild:release-9.1.0from
bazel-io:cp28684-9.1.0

Conversation

@bazel-io
Copy link
Copy Markdown
Member

Description

Reencode internal strings to Unicode strings.

Motivation

UploadManifest got this right for files, but not for the contents of output directories.

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

Closes #28684.

PiperOrigin-RevId: 871781011
Change-Id: Ia7aa2d746ea25ed47454797e62e68a93f4e575bc

Commit 0449267

### Description
Reencode internal strings to Unicode strings.

### Motivation
`UploadManifest` got this right for files, but not for the contents of output directories.

### Build API Changes

No

### Checklist

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

### Release Notes

Closes bazelbuild#28684.

PiperOrigin-RevId: 871781011
Change-Id: Ia7aa2d746ea25ed47454797e62e68a93f4e575bc
@bazel-io bazel-io requested a review from a team as a code owner February 18, 2026 11:38
@bazel-io bazel-io added team-Remote-Exec Issues and PRs for the Execution (Remote) team awaiting-review PR is awaiting review from an assigned reviewer labels Feb 18, 2026
@bazel-io bazel-io requested a review from tjgq February 18, 2026 11:38
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

The pull request correctly addresses the Unicode encoding issue for uploaded directories by re-encoding internal strings to Unicode strings. The new test case actionResult_unicodeDirectoryTree effectively verifies this fix. The changes are well-contained and directly target the identified problem.

@iancha1992 iancha1992 enabled auto-merge February 18, 2026 19:04
@iancha1992 iancha1992 added this pull request to the merge queue Feb 25, 2026
Merged via the queue into bazelbuild:release-9.1.0 with commit f24ca77 Feb 26, 2026
46 checks passed
@github-actions github-actions Bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Feb 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-Remote-Exec Issues and PRs for the Execution (Remote) team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants