Introduce --incompatible_no_target_output_group to turn down deprecated Target.output_group field.

Progress toward #6241

RELNOTES: None.
PiperOrigin-RevId: 215238373

src/main/java/com/google/devtools/build/lib/syntax/SkylarkSemantics.java

diff --git a/src/main/java/com/google/devtools/build/lib/syntax/SkylarkSemantics.java b/src/main/java/com/google/devtools/build/lib/syntax/SkylarkSemantics.java
index 2d606cc..fb1d2df 100644
--- a/src/main/java/com/google/devtools/build/lib/syntax/SkylarkSemantics.java
+++ b/src/main/java/com/google/devtools/build/lib/syntax/SkylarkSemantics.java
@@ -37,6 +37,8 @@
   public enum FlagIdentifier {
     INCOMPATIBLE_DISABLE_OBJC_PROVIDER_RESOURCES(
         SkylarkSemantics::incompatibleDisableObjcProviderResources),
+    INCOMPATIBLE_NO_TARGET_OUTPUT_GROUP(
+        SkylarkSemantics::incompatibleNoTargetOutputGroup),
     NONE(null);
 
     // Using a Function here makes the enum definitions far cleaner, and, since this is
@@ -125,6 +127,8 @@
 
   public abstract boolean incompatibleNoSupportToolsInActionInputs();
 
+  public abstract boolean incompatibleNoTargetOutputGroup();
+
   public abstract boolean incompatibleNoTransitiveLoads();
 
   public abstract boolean incompatiblePackageNameIsAFunction();
@@ -177,6 +181,7 @@
           .incompatibleGenerateJavaCommonSourceJar(false)
           .incompatibleNewActionsApi(false)
           .incompatibleNoSupportToolsInActionInputs(false)
+          .incompatibleNoTargetOutputGroup(false)
           .incompatibleNoTransitiveLoads(false)
           .incompatiblePackageNameIsAFunction(false)
           .incompatibleRangeType(false)
@@ -231,6 +236,8 @@
 
     public abstract Builder incompatibleNoSupportToolsInActionInputs(boolean value);
 
+    public abstract Builder incompatibleNoTargetOutputGroup(boolean value);
+
     public abstract Builder incompatibleNoTransitiveLoads(boolean value);
 
     public abstract Builder incompatiblePackageNameIsAFunction(boolean value);