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

[7.2.0] Fix writable sandbox path #22451

Merged
merged 1 commit into from
May 20, 2024

Conversation

bazel-io
Copy link
Member

The reverts in 3fddc7f accidentally replaced sandboxExecRoot with execRoot in the function that computes writable directories for sandbox spawn runners.

This issue was observed in MobileNativeFoundation/rules_xcodeproj#3029.

Closes #22443.

PiperOrigin-RevId: 635383877
Change-Id: I3d74bf78cf97ecd267ba90b68cf1b715850aba27

Commit b0ed4ca

The reverts in 3fddc7f accidentally replaced `sandboxExecRoot` with `execRoot` in the function that computes writable directories for sandbox spawn runners.

This issue was observed in MobileNativeFoundation/rules_xcodeproj#3029.

Closes bazelbuild#22443.

PiperOrigin-RevId: 635383877
Change-Id: I3d74bf78cf97ecd267ba90b68cf1b715850aba27
@bazel-io bazel-io requested a review from a team as a code owner May 20, 2024 09:01
@bazel-io bazel-io added team-Local-Exec Issues and PRs for the Execution (Local) team awaiting-review PR is awaiting review from an assigned reviewer labels May 20, 2024
@bazel-io bazel-io requested a review from oquenchil May 20, 2024 09:01
@keertk keertk added this pull request to the merge queue May 20, 2024
Merged via the queue into bazelbuild:release-7.2.0 with commit d31ce86 May 20, 2024
33 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Local-Exec Issues and PRs for the Execution (Local) team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants