diff options
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.java | 21 |
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()); } /** |