Delete unused flag
The flag has been replaced by the "_validation_transitive" special output group
@@ -95,12 +95,10 @@ public final class RuleConfiguredTargetBuilder {
private ImmutableSet<ActionAnalysisMetadata> actionsWithoutExtraAction = ImmutableSet.of();
private final LinkedHashSet<String> ruleImplSpecificRequiredConfigFragments =
new LinkedHashSet<>();
- private boolean propagateValidationActionOutputGroup = true;
public RuleConfiguredTargetBuilder(RuleContext ruleContext) {
this.ruleContext = ruleContext;
// Avoid building validations in analysis tests (b/143988346)
- propagateValidationActionOutputGroup = !ruleContext.getRule().isAnalysisTest();
add(LicensesProvider.class, LicensesProviderImpl.of(ruleContext));
add(VisibilityProvider.class, new VisibilityProviderImpl(ruleContext.getVisibility()));
}
@@ -161,7 +159,7 @@ public final class RuleConfiguredTargetBuilder {
.getAllArtifacts());
}
- if (propagateValidationActionOutputGroup) {
+ if (propagateValidationActionOutputGroup()) {
propagateTransitiveValidationOutputGroups();
}
@@ -278,6 +276,10 @@ public final class RuleConfiguredTargetBuilder {
generatingActions.getArtifactsByOutputLabel());
}
+ private boolean propagateValidationActionOutputGroup() {
+ return !ruleContext.getRule().isAnalysisTest();
+ }
+
/** Actually process */
private void handleAllowlistChecker(AllowlistChecker allowlistChecker) {
if (allowlistChecker.attributeSetTrigger() != null
@@ -652,12 +654,6 @@ public final class RuleConfiguredTargetBuilder {
return this;
}
- /** Sets whether to propagate the validation actions output group. This is true by default. */
- public RuleConfiguredTargetBuilder setPropagateValidationActionOutputGroup(boolean propagate) {
- this.propagateValidationActionOutputGroup = propagate;
- return this;
- }
-
private NestedSetBuilder<Artifact> getOutputGroupBuilder(String name) {
NestedSetBuilder<Artifact> result = outputGroupBuilders.get(name);
if (result != null) {
