aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/main/java/com/google/devtools/build/lib/sandbox/LinuxSandboxedStrategy.java
diff options
context:
space:
mode:
authorGravatar jmmv <jmmv@google.com>2018-03-23 02:47:44 -0700
committerGravatar Copybara-Service <copybara-piper@google.com>2018-03-23 02:49:10 -0700
commit7a51a7c44df334f8e67a793788caf520b5127db4 (patch)
tree7e50ac2fd8ba2a1a2830d32be9ec9e3c53fe26cb /src/main/java/com/google/devtools/build/lib/sandbox/LinuxSandboxedStrategy.java
parent3b9cf80c3cadce94ca4a7aa1a3f5c8756e4ac9e4 (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.java30
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");