Remove RuleConfiguredTarget#getMandatoryStampFiles().

This is apparently unused.

--
MOS_MIGRATED_REVID=120317405

2 files changed, 1 insertions, 28 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/RuleConfiguredTarget.java b/src/main/java/com/google/devtools/build/lib/analysis/RuleConfiguredTarget.java
index c70b5b0843..9773cc1062 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/RuleConfiguredTarget.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/RuleConfiguredTarget.java
@@ -18,7 +18,6 @@ import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ImmutableMap;
 import com.google.common.collect.Iterables;
 import com.google.common.collect.UnmodifiableIterator;
-import com.google.devtools.build.lib.actions.Artifact;
 import com.google.devtools.build.lib.analysis.config.ConfigMatchingProvider;
 import com.google.devtools.build.lib.analysis.config.RunUnder;
 import com.google.devtools.build.lib.packages.OutputFile;
@@ -51,12 +50,10 @@ public final class RuleConfiguredTarget extends AbstractConfiguredTarget {
   }
 
   private final ImmutableMap<Class<? extends TransitiveInfoProvider>, Object> providers;
-  private final ImmutableList<Artifact> mandatoryStampFiles;
   private final Set<ConfigMatchingProvider> configConditions;
   private final ImmutableList<ConfiguredAspect> configuredAspects;
 
   RuleConfiguredTarget(RuleContext ruleContext,
-      ImmutableList<Artifact> mandatoryStampFiles,
       ImmutableMap<String, Object> skylarkProviders,
       Map<Class<? extends TransitiveInfoProvider>, TransitiveInfoProvider> providers) {
     super(ruleContext);
@@ -78,7 +75,6 @@ public final class RuleConfiguredTarget extends AbstractConfiguredTarget {
     providerBuilder.put(SkylarkProviders.class, new SkylarkProviders(skylarkProviders));
 
     this.providers = ImmutableMap.copyOf(providerBuilder);
-    this.mandatoryStampFiles = mandatoryStampFiles;
     this.configConditions = ruleContext.getConfigConditions();
     this.configuredAspects = ImmutableList.of();
 
@@ -190,7 +186,6 @@ public final class RuleConfiguredTarget extends AbstractConfiguredTarget {
       }
       this.providers = builder.build();
     }
-    this.mandatoryStampFiles = base.mandatoryStampFiles;
     this.configConditions = base.configConditions;
     this.configuredAspects = ImmutableList.copyOf(aspects);
   }
@@ -246,10 +241,6 @@ public final class RuleConfiguredTarget extends AbstractConfiguredTarget {
     return getProvider(SkylarkProviders.class).getValue(providerKey);
   }
 
-  public ImmutableList<Artifact> getMandatoryStampFiles() {
-    return mandatoryStampFiles;
-  }
-
   @Override
   public final Rule getTarget() {
     return (Rule) super.getTarget();
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/RuleConfiguredTargetBuilder.java b/src/main/java/com/google/devtools/build/lib/analysis/RuleConfiguredTargetBuilder.java
index 25438ec2e1..7be598c2b0 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/RuleConfiguredTargetBuilder.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/RuleConfiguredTargetBuilder.java
@@ -20,7 +20,6 @@ import com.google.common.collect.ImmutableMap;
 import com.google.common.collect.ImmutableSet;
 import com.google.devtools.build.lib.actions.Action;
 import com.google.devtools.build.lib.actions.Artifact;
-import com.google.devtools.build.lib.analysis.ExtraActionArtifactsProvider.ExtraArtifactSet;
 import com.google.devtools.build.lib.analysis.LicensesProvider.TargetLicense;
 import com.google.devtools.build.lib.analysis.config.BuildConfiguration;
 import com.google.devtools.build.lib.analysis.constraints.ConstraintSemantics;
@@ -63,7 +62,6 @@ public final class RuleConfiguredTargetBuilder {
   private NestedSet<Artifact> filesToBuild = NestedSetBuilder.emptySet(Order.STABLE_ORDER);
   private RunfilesSupport runfilesSupport;
   private Artifact executable;
-  private ImmutableList<Artifact> mandatoryStampFiles;
   private ImmutableSet<Action> actionsWithoutExtraAction = ImmutableSet.of();
 
   public RuleConfiguredTargetBuilder(RuleContext ruleContext) {
@@ -115,12 +113,6 @@ public final class RuleConfiguredTargetBuilder {
 
     ExtraActionArtifactsProvider extraActionsProvider =
         createExtraActionProvider(actionsWithoutExtraAction, ruleContext);
-    if (mandatoryStampFiles != null && !mandatoryStampFiles.isEmpty()) {
-      extraActionsProvider = ExtraActionArtifactsProvider.create(
-          extraActionsProvider.getExtraActionArtifacts(),
-          NestedSetBuilder.fromNestedSet(extraActionsProvider.getTransitiveExtraActionArtifacts())
-              .add(ExtraArtifactSet.of(ruleContext.getLabel(), mandatoryStampFiles)).build());
-    }
     add(ExtraActionArtifactsProvider.class, extraActionsProvider);
 
     if (!outputGroupBuilders.isEmpty()) {
@@ -133,8 +125,7 @@ public final class RuleConfiguredTargetBuilder {
     }
 
 
-    return new RuleConfiguredTarget(
-        ruleContext, mandatoryStampFiles, skylarkProviders.build(), providers);
+    return new RuleConfiguredTarget(ruleContext, skylarkProviders.build(), providers);
   }
 
   /**
@@ -330,15 +321,6 @@ public final class RuleConfiguredTargetBuilder {
   }
 
   /**
-   * Set the mandatory stamp files.
-   */
-  public RuleConfiguredTargetBuilder setMandatoryStampFiles(ImmutableList<Artifact> files) {
-    this.mandatoryStampFiles = files;
-    return this;
-  }
-
-
-  /**
    * Set the extra action pseudo actions.
    */
   public RuleConfiguredTargetBuilder setActionsWithoutExtraAction(ImmutableSet<Action> actions) {