Remove RuleConfiguredTarget#getMandatoryStampFiles().
This is apparently unused.
--
MOS_MIGRATED_REVID=120317405
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 c70b5b0..9773cc1 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.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 @@
}
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 @@
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 @@
}
this.providers = builder.build();
}
- this.mandatoryStampFiles = base.mandatoryStampFiles;
this.configConditions = base.configConditions;
this.configuredAspects = ImmutableList.copyOf(aspects);
}
@@ -246,10 +241,6 @@
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 25438ec..7be598c 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.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 @@
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 @@
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 @@
}
- return new RuleConfiguredTarget(
- ruleContext, mandatoryStampFiles, skylarkProviders.build(), providers);
+ return new RuleConfiguredTarget(ruleContext, skylarkProviders.build(), providers);
}
/**
@@ -330,15 +321,6 @@
}
/**
- * 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) {
