Revert "Fix repositories to use 'BUILD.bazel' as the name for the symlinked/new build file, which has less changes of causing conflicts. Fixes #2226."

This reverts commit 2f31d60d1454cd324196a5159262c1ba32eabda8.
Re-open #2226.

--
Change-Id: I7f223df4bb239daf8c306955e92f817310e6cd4f
Reviewed-on: https://cr.bazel.build/7950
PiperOrigin-RevId: 142257579
MOS_MIGRATED_REVID=142257579

4 files changed, 10 insertions, 9 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/bazel/repository/JarDecompressor.java b/src/main/java/com/google/devtools/build/lib/bazel/repository/JarDecompressor.java
index 703c4a32dd..665692d8d9 100644
--- a/src/main/java/com/google/devtools/build/lib/bazel/repository/JarDecompressor.java
+++ b/src/main/java/com/google/devtools/build/lib/bazel/repository/JarDecompressor.java
@@ -37,8 +37,8 @@ public class JarDecompressor implements Decompressor {
   /**
    * The .jar can be used compressed, so this just exposes it in a way Bazel can use.
    *
-   * <p>It moves the jar from some-name/x/y/z/foo.jar to some-name/jar/foo.jar and creates a
-   * BUILD.bazel file containing one entry: the .jar.
+   * <p>It moves the jar from some-name/x/y/z/foo.jar to some-name/jar/foo.jar and creates a BUILD
+   * file containing one entry: the .jar.
    */
   @Override
   @Nullable
@@ -60,12 +60,12 @@ public class JarDecompressor implements Decompressor {
       if (!jarSymlink.exists()) {
         jarSymlink.createSymbolicLink(descriptor.archivePath());
       }
-      // external/some-name/repository/jar/BUILD.bazel defines the //jar target.
-      Path buildFile = jarDirectory.getRelative("BUILD.bazel");
+      // external/some-name/repository/jar/BUILD defines the //jar target.
+      Path buildFile = jarDirectory.getRelative("BUILD");
       FileSystemUtils.writeLinesAs(
           buildFile,
           Charset.forName("UTF-8"),
-          "# DO NOT EDIT: automatically generated BUILD.bazel file for "
+          "# DO NOT EDIT: automatically generated BUILD file for "
               + descriptor.targetKind()
               + " rule "
               + descriptor.targetName(),
diff --git a/src/main/java/com/google/devtools/build/lib/rules/repository/NewRepositoryBuildFileHandler.java b/src/main/java/com/google/devtools/build/lib/rules/repository/NewRepositoryBuildFileHandler.java
index 76868d77e4..658c957690 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/repository/NewRepositoryBuildFileHandler.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/repository/NewRepositoryBuildFileHandler.java
@@ -106,7 +106,7 @@ public class NewRepositoryBuildFileHandler {
    */
   public void finishBuildFile(Path outputDirectory) throws RepositoryFunctionException {
     if (buildFileValue != null) {
-      // Link x/BUILD to <build_root>/x.BUILD.bazel.
+      // Link x/BUILD to <build_root>/x.BUILD.
       symlinkBuildFile(buildFileValue, outputDirectory);
     } else if (buildFileContent != null) {
       RepositoryFunction.writeBuildFile(outputDirectory, buildFileContent);
@@ -204,7 +204,7 @@ public class NewRepositoryBuildFileHandler {
    */
   private void symlinkBuildFile(
       FileValue buildFileValue, Path outputDirectory) throws RepositoryFunctionException {
-    Path buildFilePath = outputDirectory.getRelative("BUILD.bazel");
+    Path buildFilePath = outputDirectory.getRelative("BUILD");
     RepositoryFunction.createSymbolicLink(buildFilePath, buildFileValue.realRootedPath().asPath());
   }
 }
diff --git a/src/main/java/com/google/devtools/build/lib/rules/repository/RepositoryFunction.java b/src/main/java/com/google/devtools/build/lib/rules/repository/RepositoryFunction.java
index 03d886e4fa..c717905204 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/repository/RepositoryFunction.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/repository/RepositoryFunction.java
@@ -197,7 +197,7 @@ public abstract class RepositoryFunction {
 
   protected static RepositoryDirectoryValue writeBuildFile(
       Path repositoryDirectory, String contents) throws RepositoryFunctionException {
-    Path buildFilePath = repositoryDirectory.getRelative("BUILD.bazel");
+    Path buildFilePath = repositoryDirectory.getRelative("BUILD");
     try {
       // The repository could have an existing BUILD file that's either a regular file (for remote
       // repositories) or a symlink (for local repositories). Either way, we want to remove it and
diff --git a/src/test/java/com/google/devtools/build/lib/rules/repository/JarDecompressorTest.java b/src/test/java/com/google/devtools/build/lib/rules/repository/JarDecompressorTest.java
index 37cfa8daa3..c4596684d8 100644
--- a/src/test/java/com/google/devtools/build/lib/rules/repository/JarDecompressorTest.java
+++ b/src/test/java/com/google/devtools/build/lib/rules/repository/JarDecompressorTest.java
@@ -53,7 +53,7 @@ public class JarDecompressorTest {
     Path outputDir = DecompressorValue.decompress(descriptorBuilder.build());
     assertThat(outputDir.exists()).isTrue();
     String buildContent =
-        new String(FileSystemUtils.readContentAsLatin1(outputDir.getRelative("jar/BUILD.bazel")));
+        new String(FileSystemUtils.readContentAsLatin1(outputDir.getRelative("jar/BUILD")));
     assertThat(buildContent).contains("java_import");
     assertThat(buildContent).contains("filegroup");
   }
@@ -66,4 +66,5 @@ public class JarDecompressorTest {
         FileSystemUtils.readContentAsLatin1(outputDir.getRelative("WORKSPACE")));
     assertThat(workspaceContent).contains("workspace(name = \"tester\")");
   }
+
 }