commit ea11fc54f4224094bc3793eec1e2eaf7d397c65f
author Ulf Adams <ulfjack@google.com> Tue Aug 04 14:23:58 2015 +0000
committer John Field <jfield@google.com> Tue Aug 04 20:25:51 2015 +0000
tree fc2a5b84adedbed150cc160b2a044fadd8a12593
parent d1367eae6ab5824b5b9f23f2d93e764e1e47de84

Add more configuration fragment annotations, and a runtime check.

--
MOS_MIGRATED_REVID=99819719

src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java b/src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java
index fad5cd4..92b3dbd 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java
@@ -259,12 +259,15 @@
   public <T extends Fragment> T getFragment(Class<T> fragment) {
     // TODO(bazel-team): The fragments can also be accessed directly through BuildConfiguration.
     // Can we lock that down somehow?
-    Preconditions.checkArgument(
-        rule.getRuleClassObject().isLegalConfigurationFragment(fragment),
+    Preconditions.checkArgument(isLegalFragment(fragment),
         "%s does not have access to %s", rule.getRuleClass(), fragment);
     return getConfiguration().getFragment(fragment);
   }
 
+  public <T extends Fragment> boolean isLegalFragment(Class<T> fragment) {
+    return rule.getRuleClassObject().isLegalConfigurationFragment(fragment);
+  }
+
   @Override
   public ArtifactOwner getOwner() {
     return getAnalysisEnvironment().getOwner();