diff options
author | John Cater <jcater@google.com> | 2016-12-16 15:22:25 +0000 |
---|---|---|
committer | John Cater <jcater@google.com> | 2016-12-16 15:37:57 +0000 |
commit | fa3f55a8785c28253b2f836ddc84e994e48edd7f (patch) | |
tree | 2ee5212cfdc77a1c01c65375699f8cc9b9dea977 /src/main/java/com/google | |
parent | b21cfa12b83072bcb6d7e700542e7051a7d21bce (diff) |
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
Diffstat (limited to 'src/main/java/com/google')
3 files changed, 8 insertions, 8 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 |