Remove some dead AbstractAction constructors

The constructors which don't accept an action environment should not be called,
with the exception of subclasses that don't run any subprocesses, and therefore
don't need an environment.

PiperOrigin-RevId: 197706734

1 files changed, 0 insertions, 34 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/actions/AbstractAction.java b/src/main/java/com/google/devtools/build/lib/actions/AbstractAction.java
index b52bc2f8d7..034f6c2670 100644
--- a/src/main/java/com/google/devtools/build/lib/actions/AbstractAction.java
+++ b/src/main/java/com/google/devtools/build/lib/actions/AbstractAction.java
@@ -133,40 +133,6 @@ public abstract class AbstractAction implements Action, ActionApi {
         env);
   }
 
-  /**
-   * Construct an abstract action with the specified tools, inputs and outputs;
-   */
-  protected AbstractAction(
-      ActionOwner owner,
-      Iterable<Artifact> tools,
-      Iterable<Artifact> inputs,
-      Iterable<Artifact> outputs) {
-    this(owner, tools, inputs, EmptyRunfilesSupplier.INSTANCE, outputs, ActionEnvironment.EMPTY);
-  }
-
-  protected AbstractAction(
-      ActionOwner owner,
-      Iterable<Artifact> inputs,
-      RunfilesSupplier runfilesSupplier,
-      Iterable<Artifact> outputs) {
-    this(
-        owner,
-        /*tools=*/ImmutableList.of(),
-        inputs,
-        runfilesSupplier,
-        outputs,
-        ActionEnvironment.EMPTY);
-  }
-
-  protected AbstractAction(
-      ActionOwner owner,
-      Iterable<Artifact> tools,
-      Iterable<Artifact> inputs,
-      RunfilesSupplier runfilesSupplier,
-      Iterable<Artifact> outputs) {
-    this(owner, tools, inputs, runfilesSupplier, outputs, ActionEnvironment.EMPTY);
-  }
-
   protected AbstractAction(
       ActionOwner owner,
       Iterable<Artifact> tools,