Revert back to the old behavior of not creating a proto source root …

…for generated .proto files.

    It was meant to be a compatible change, but alas, people seemed to depend on
    the exact layout of the output tree. Will flip in 1.0 . Tracking bug:

    bazelbuild/bazel#9215

    RELNOTES: None.
    PiperOrigin-RevId: 264581654

dataset/GitHub_Java/bazelbuild.bazel/src/main/java/com/google/devtools/build/lib/rules/proto/ProtoConfiguration.java

@@ -42,7 +42,7 @@ public class ProtoConfiguration extends Fragment implements ProtoConfigurationAp
   public static class Options extends FragmentOptions {
     @Option(
         name = "incompatible_generated_protos_in_virtual_imports",
-        defaultValue = "true",
+        defaultValue = "false",
         documentationCategory = OptionDocumentationCategory.UNDOCUMENTED,
         effectTags = {OptionEffectTag.LOADING_AND_ANALYSIS},
         metadataTags = {

dataset/GitHub_Java/bazelbuild.bazel/src/test/java/com/google/devtools/build/lib/rules/proto/BazelProtoLibraryTest.java

@@ -370,16 +370,13 @@ public void testExternalRepoWithGeneratedProto() throws Exception {
         TestConstants.LOAD_PROTO_LIBRARY,
         "proto_library(name='a', srcs=['a.proto'], deps=['@foo//x:x'])");
 
-    String genfiles = getTargetConfiguration().getGenfilesFragment().toString();
     ConfiguredTarget a = getConfiguredTarget("//a:a");
     ProtoInfo aInfo = a.get(ProtoInfo.PROVIDER);
-    assertThat(aInfo.getTransitiveProtoSourceRoots())
-        .containsExactly(".", genfiles + "/external/foo/x/_virtual_imports/x");
+    assertThat(aInfo.getTransitiveProtoSourceRoots()).containsExactly(".", "external/foo");
 
     ConfiguredTarget x = getConfiguredTarget("@foo//x:x");
     ProtoInfo xInfo = x.get(ProtoInfo.PROVIDER);
-    assertThat(xInfo.getTransitiveProtoSourceRoots())
-        .containsExactly(genfiles + "/external/foo/x/_virtual_imports/x");
+    assertThat(xInfo.getTransitiveProtoSourceRoots()).containsExactly("external/foo");
   }
 
   @Test