aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/main/java/com/google/devtools/build/lib/analysis/ExtraActionUtils.java
diff options
context:
space:
mode:
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/analysis/ExtraActionUtils.java')
-rw-r--r--src/main/java/com/google/devtools/build/lib/analysis/ExtraActionUtils.java21
1 files changed, 5 insertions, 16 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/ExtraActionUtils.java b/src/main/java/com/google/devtools/build/lib/analysis/ExtraActionUtils.java
index f59281042c..520f40e0ba 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/ExtraActionUtils.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/ExtraActionUtils.java
@@ -44,8 +44,7 @@ class ExtraActionUtils {
* bookkeeping.
*/
static ExtraActionArtifactsProvider createExtraActionProvider(
- Set<Action> actionsWithoutExtraAction, List<Artifact> mandatoryStampFiles,
- RuleContext ruleContext) {
+ Set<Action> actionsWithoutExtraAction, RuleContext ruleContext) {
BuildConfiguration configuration = ruleContext.getConfiguration();
if (configuration.isHostConfiguration()) {
return ExtraActionArtifactsProvider.EMPTY;
@@ -76,22 +75,12 @@ class ExtraActionUtils {
}
// Add extra action artifacts from dependencies
- for (TransitiveInfoCollection dep : ruleContext.getConfiguredTargetMap().values()) {
- ExtraActionArtifactsProvider provider =
- dep.getProvider(ExtraActionArtifactsProvider.class);
- if (provider != null) {
- builder.addTransitive(provider.getTransitiveExtraActionArtifacts());
- }
- }
-
- if (mandatoryStampFiles != null && !mandatoryStampFiles.isEmpty()) {
- builder.add(ExtraArtifactSet.of(ruleContext.getLabel(), mandatoryStampFiles));
+ for (ExtraActionArtifactsProvider provider : AnalysisUtils.getProviders(
+ ruleContext.getConfiguredTargetMap().values(), ExtraActionArtifactsProvider.class)) {
+ builder.addTransitive(provider.getTransitiveExtraActionArtifacts());
}
- if (extraActionArtifacts.isEmpty() && builder.isEmpty()) {
- return ExtraActionArtifactsProvider.EMPTY;
- }
- return new ExtraActionArtifactsProvider(extraActionArtifacts, builder.build());
+ return ExtraActionArtifactsProvider.create(extraActionArtifacts, builder.build());
}
/**