commit a6c88966a5dc7713df4ec64bbf935bc5023e2f9d
author Greg Estren <gregce@google.com> Mon Sep 28 19:35:18 2015 +0000
committer Florian Weikert <fwe@google.com> Wed Sep 30 09:32:05 2015 +0000
tree 95a2905a2a3c55fe54e4dc1559bec6a41e254bfe
parent 1743660d416b00084a41b20ea383304cafad3600

Don't crash when building environment groups directly
(following PackageGroup's precedent).

Also cleanup: generalize the pattern by which we determine
non-configurable target types.

--
MOS_MIGRATED_REVID=104125803

src/main/java/com/google/devtools/build/lib/packages/PackageGroup.java

diff --git a/src/main/java/com/google/devtools/build/lib/packages/PackageGroup.java b/src/main/java/com/google/devtools/build/lib/packages/PackageGroup.java
index d7fe3df..3695e36 100644
--- a/src/main/java/com/google/devtools/build/lib/packages/PackageGroup.java
+++ b/src/main/java/com/google/devtools/build/lib/packages/PackageGroup.java
@@ -146,6 +146,11 @@
     return ConstantRuleVisibility.PUBLIC;
   }
 
+  @Override
+  public boolean isConfigurable() {
+    return false;
+  }
+
   public static String targetKind() {
     return "package group";
   }