diff options
author | 2018-01-08 00:45:57 -0800 | |
---|---|---|
committer | 2018-01-08 00:48:00 -0800 | |
commit | 6ab93a6048260671206622527c89192b1a86284b (patch) | |
tree | 8b6c2425eb5b7125bbf2a418d80101328053448b /src/main/java/com/google/devtools/build/lib/sandbox/DarwinSandboxedSpawnRunner.java | |
parent | ecb481d9afad8863d95aa53a24dc9a75bd204e8f (diff) |
sandbox runners: always use updated environment
The sandboxed Spawn runners already used to call
LocalEnvProvider.rewriteEnvironment in order to
add TMPDIR for example, but didn't consistently
use this rewritten environment everywhere, and
used the Spawn's original environment instead.
Change-Id: Ifbede6ac0f14b83c617dcbcd85edfae88718b157
PiperOrigin-RevId: 181130506
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/sandbox/DarwinSandboxedSpawnRunner.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/sandbox/DarwinSandboxedSpawnRunner.java | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/sandbox/DarwinSandboxedSpawnRunner.java b/src/main/java/com/google/devtools/build/lib/sandbox/DarwinSandboxedSpawnRunner.java index 91b3f2370b..6199b5d27f 100644 --- a/src/main/java/com/google/devtools/build/lib/sandbox/DarwinSandboxedSpawnRunner.java +++ b/src/main/java/com/google/devtools/build/lib/sandbox/DarwinSandboxedSpawnRunner.java @@ -222,12 +222,11 @@ final class DarwinSandboxedSpawnRunner extends AbstractSandboxSpawnRunner { // name unique (like we have to with standalone execution strategy). Path tmpDir = sandboxExecRoot.getRelative("tmp"); - Map<String, String> spawnEnvironment = + Map<String, String> environment = localEnvProvider.rewriteLocalEnv(spawn.getEnvironment(), execRoot, tmpDir, productName); final HashSet<Path> writableDirs = new HashSet<>(alwaysWritableDirs); - ImmutableSet<Path> extraWritableDirs = - getWritableDirs(sandboxExecRoot, spawnEnvironment, tmpDir); + ImmutableSet<Path> extraWritableDirs = getWritableDirs(sandboxExecRoot, environment, tmpDir); writableDirs.addAll(extraWritableDirs); ImmutableSet<PathFragment> outputs = SandboxHelpers.getOutputFiles(spawn); @@ -259,9 +258,6 @@ final class DarwinSandboxedSpawnRunner extends AbstractSandboxSpawnRunner { .addAll(processWrapperCommandLineBuilder.build()) .build(); - Map<String, String> environment = - localEnvProvider.rewriteLocalEnv(spawn.getEnvironment(), execRoot, tmpDir, productName); - boolean allowNetworkForThisSpawn = allowNetwork || Spawns.requiresNetwork(spawn); SandboxedSpawn sandbox = new SymlinkedSandboxedSpawn( |