diff options
author | Ulf Adams <ulfjack@google.com> | 2015-08-07 11:32:43 +0000 |
---|---|---|
committer | Dmitry Lomov <dslomov@google.com> | 2015-08-10 10:09:15 +0000 |
commit | 84a1278f24e350f57ab30142331b8bd698266cb2 (patch) | |
tree | c8726ab43eb36328d00e338d1da2dbc88e049deb /src/main/java/com/google/devtools/build/lib/packages | |
parent | d2ae11fb4524e9fe8897c43f5aa2307b5ea531e7 (diff) |
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
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/packages')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/packages/RuleClass.java | 2 |
1 files changed, 1 insertions, 1 deletions
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; } |