Skip to content

Commit d31ce86

Browse files
bazel-iofmeum
andauthored
[7.2.0] Fix writable sandbox path (bazelbuild#22451)
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 Commit bazelbuild@b0ed4ca Co-authored-by: Fabian Meumertzheim <[email protected]>
1 parent b229d83 commit d31ce86

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/main/java/com/google/devtools/build/lib/sandbox/AbstractSandboxSpawnRunner.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -372,7 +372,7 @@ protected ImmutableSet<Path> getWritableDirs(Path sandboxExecRoot, Map<String, S
372372
// On Windows, sandboxExecRoot is actually the main execroot. We will specify
373373
// exactly which output path is writable.
374374
if (OS.getCurrent() != OS.WINDOWS) {
375-
writablePaths.add(execRoot);
375+
writablePaths.add(sandboxExecRoot);
376376
}
377377

378378
String testTmpdir = env.get("TEST_TMPDIR");

0 commit comments

Comments
 (0)