Skip to content

Commit

Permalink
Fix writable sandbox path
Browse files Browse the repository at this point in the history
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
  • Loading branch information
fmeum authored and bazel-io committed May 20, 2024
1 parent ca9c753 commit 461da24
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -372,7 +372,7 @@ protected ImmutableSet<Path> getWritableDirs(Path sandboxExecRoot, Map<String, S
// On Windows, sandboxExecRoot is actually the main execroot. We will specify
// exactly which output path is writable.
if (OS.getCurrent() != OS.WINDOWS) {
writablePaths.add(execRoot);
writablePaths.add(sandboxExecRoot);
}

String testTmpdir = env.get("TEST_TMPDIR");
Expand Down

0 comments on commit 461da24

Please sign in to comment.