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
diff --git a/src/main/java/com/google/devtools/build/lib/packages/EnvironmentGroup.java b/src/main/java/com/google/devtools/build/lib/packages/EnvironmentGroup.java
index a5b977c..0536dc3 100644
--- a/src/main/java/com/google/devtools/build/lib/packages/EnvironmentGroup.java
+++ b/src/main/java/com/google/devtools/build/lib/packages/EnvironmentGroup.java
@@ -310,6 +310,11 @@
return ConstantRuleVisibility.PRIVATE; // No rule should be referencing an environment_group.
}
+ @Override
+ public boolean isConfigurable() {
+ return false;
+ }
+
public static String targetKind() {
return "environment group";
}