diff options
author | 2018-03-23 02:47:44 -0700 | |
---|---|---|
committer | 2018-03-23 02:49:10 -0700 | |
commit | 7a51a7c44df334f8e67a793788caf520b5127db4 (patch) | |
tree | 7e50ac2fd8ba2a1a2830d32be9ec9e3c53fe26cb /src/main/java/com/google/devtools/build/lib/sandbox/LinuxSandboxedStrategy.java | |
parent | 3b9cf80c3cadce94ca4a7aa1a3f5c8756e4ac9e4 (diff) |
Remove unnecessary product name and constructors from sandboxed spawn runners.
The product name was previously propagated to all sandboxed spawn runners
unconditionally, even when the majority did not need it. Also, these
same spawn runners had obsolete constructors that were not used at all or
were only used in tests. Remove both for simplicity.
RELNOTES: None.
PiperOrigin-RevId: 190195305
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/sandbox/LinuxSandboxedStrategy.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/sandbox/LinuxSandboxedStrategy.java | 30 |
1 files changed, 0 insertions, 30 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/sandbox/LinuxSandboxedStrategy.java b/src/main/java/com/google/devtools/build/lib/sandbox/LinuxSandboxedStrategy.java index 360fcbbce4..32997347b8 100644 --- a/src/main/java/com/google/devtools/build/lib/sandbox/LinuxSandboxedStrategy.java +++ b/src/main/java/com/google/devtools/build/lib/sandbox/LinuxSandboxedStrategy.java @@ -42,46 +42,16 @@ public final class LinuxSandboxedStrategy extends AbstractSpawnStrategy { } /** - * Creates a sandboxed spawn runner that uses the {@code linux-sandbox} tool. If a spawn exceeds - * its timeout, then it will be killed instantly. - * - * @param cmdEnv the command environment to use - * @param sandboxBase path to the sandbox base directory - * @param productName the product name to use - */ - static LinuxSandboxedSpawnRunner create( - CommandEnvironment cmdEnv, Path sandboxBase, String productName) throws IOException { - return create(cmdEnv, sandboxBase, productName, Optional.empty()); - } - - /** - * Creates a sandboxed spawn runner that uses the {@code linux-sandbox} tool. If a spawn exceeds - * its timeout, then it will be killed after the specified delay. - * - * @param cmdEnv the command environment to use - * @param sandboxBase path to the sandbox base directory - * @param productName the product name to use - * @param timeoutKillDelay an additional grace period before killing timing out commands - */ - static LinuxSandboxedSpawnRunner create( - CommandEnvironment cmdEnv, Path sandboxBase, String productName, Duration timeoutKillDelay) - throws IOException { - return create(cmdEnv, sandboxBase, productName, Optional.of(timeoutKillDelay)); - } - - /** * Creates a sandboxed spawn runner that uses the {@code linux-sandbox} tool. * * @param cmdEnv the command environment to use * @param sandboxBase path to the sandbox base directory - * @param productName the product name to use * @param timeoutKillDelay an optional, additional grace period before killing timing out * commands. If not present, then no grace period is used and commands are killed instantly. */ static LinuxSandboxedSpawnRunner create( CommandEnvironment cmdEnv, Path sandboxBase, - String productName, Optional<Duration> timeoutKillDelay) throws IOException { Path inaccessibleHelperFile = sandboxBase.getRelative("inaccessibleHelperFile"); |