Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correctly format non-ASCII paths, arguments, and env vars in aquery o… #15447

Merged
merged 1 commit into from
May 10, 2022
Merged

Correctly format non-ASCII paths, arguments, and env vars in aquery o… #15447

merged 1 commit into from
May 10, 2022

Conversation

ckolli5
Copy link

@ckolli5 ckolli5 commented May 10, 2022

…utput and remote execution protocol.

For historical reasons, Bazel internally encodes non-ASCII in BUILD/bzl files by taking individual input bytes (assumed to be UTF-8) and storing them in a String as the corresponding Latin1 characters. This encoding must be undone whenever these strings escape to the outside world.

Fixes #14381.

Closes #15333.

PiperOrigin-RevId: 445941013

…utput and remote execution protocol.

For historical reasons, Bazel internally encodes non-ASCII in BUILD/bzl files by taking individual input bytes (assumed to be UTF-8) and storing them in a String as the corresponding Latin1 characters. This encoding must be undone whenever these strings escape to the outside world.

Fixes #14381.

Closes #15333.

PiperOrigin-RevId: 445941013
@ckolli5 ckolli5 merged commit 65c1a33 into bazelbuild:release-5.2.0 May 10, 2022
coeuvre added a commit to coeuvre/bazel that referenced this pull request May 10, 2022
…aquery output and remote execution protocol. (bazelbuild#15447)"

This reverts commit 65c1a33.
coeuvre added a commit to coeuvre/bazel that referenced this pull request May 10, 2022
…aquery output and remote execution protocol. (bazelbuild#15447)"

This reverts commit 65c1a33.
@ShreeM01 ShreeM01 added the team-Remote-Exec Issues and PRs for the Execution (Remote) team label Dec 5, 2022
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.

3 participants