From 84a1278f24e350f57ab30142331b8bd698266cb2 Mon Sep 17 00:00:00 2001 From: Ulf Adams Date: Fri, 7 Aug 2015 11:32:43 +0000 Subject: Annotate every single rule with required configuration fragments. It's not locked down yet as Skylark rules can't declare required fragments. -- MOS_MIGRATED_REVID=100113647 --- src/main/java/com/google/devtools/build/lib/packages/RuleClass.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/main/java/com/google/devtools/build/lib/packages') diff --git a/src/main/java/com/google/devtools/build/lib/packages/RuleClass.java b/src/main/java/com/google/devtools/build/lib/packages/RuleClass.java index e2bd958b48..2a48d87f42 100644 --- a/src/main/java/com/google/devtools/build/lib/packages/RuleClass.java +++ b/src/main/java/com/google/devtools/build/lib/packages/RuleClass.java @@ -1176,7 +1176,7 @@ public final class RuleClass { */ public boolean isLegalConfigurationFragment(Class configurationFragment) { // For now, we allow all rules that don't declare allowed fragments to access any fragment. - // TODO(bazel-team): Declare fragment dependencies for all rules and remove this. + // TODO(bazel-team): All built-in rules declare fragments, but Skylark rules don't. if (requiredConfigurationFragments.isEmpty()) { return true; } -- cgit v1.2.3