From 3e87c626ed76536420aa06e4c258209b32bb76e0 Mon Sep 17 00:00:00 2001 From: Ulf Adams Date: Tue, 13 Jun 2017 10:53:13 +0200 Subject: Update BUILD files We're currently maintaining two sets of BUILD files; one at Google, and one in the Git repository. We'd like to not do that. This change makes some of the Bazel BUILD files more closely match their counterparts, in preparation for removing the internal ones. Closes #3094. PiperOrigin-RevId: 158820490 --- BUILD | 44 +++++++++++----------- .../java/com/google/devtools/build/buildjar/BUILD | 5 +-- .../google/devtools/build/buildjar/genclass/BUILD | 4 +- .../google/devtools/build/buildjar/jarhelper/BUILD | 4 +- .../com/google/devtools/build/java/turbine/BUILD | 16 +++++--- .../google/devtools/build/java/turbine/javac/BUILD | 4 +- .../com/google/devtools/build/java/bazel/BUILD | 4 ++ .../com/google/devtools/build/java/turbine/BUILD | 13 ++++--- .../google/devtools/build/java/turbine/javac/BUILD | 1 - .../com/google/testing/junit/runner/BUILD | 3 +- .../com/google/testing/junit/runner/testbed/BUILD | 2 - src/java_tools/singlejar/BUILD | 3 ++ 12 files changed, 56 insertions(+), 47 deletions(-) diff --git a/BUILD b/BUILD index 4680429dcc..5b7f88b7bf 100644 --- a/BUILD +++ b/BUILD @@ -1,7 +1,30 @@ +# Bazel - Google's Build System + package(default_visibility = ["//scripts/release:__pkg__"]) exports_files(["LICENSE"]) +filegroup( + name = "srcs", + srcs = glob( + ["*"], + exclude = [ + "bazel-*", # convenience symlinks + "out", # IntelliJ with setup-intellij.sh + "output", # output of compile.sh + ".*", # mainly .git* files + ], + ) + [ + "//examples:srcs", + "//scripts:srcs", + "//site:srcs", + "//src:srcs", + "//tools:srcs", + "//third_party:srcs", + ], + visibility = ["//visibility:private"], +) + filegroup( name = "git", srcs = glob([".git/**"]), @@ -29,27 +52,6 @@ filegroup( ], ) -filegroup( - name = "srcs", - srcs = glob( - ["*"], - exclude = [ - "bazel-*", # convenience symlinks - "out", # IntelliJ with setup-intellij.sh - "output", # output of compile.sh - ".*", # mainly .git* files - ], - ) + [ - "//examples:srcs", - "//scripts:srcs", - "//site:srcs", - "//src:srcs", - "//tools:srcs", - "//third_party:srcs", - ], - visibility = ["//visibility:private"], -) - filegroup( name = "bootstrap-derived-java-srcs", srcs = glob(["derived/**/*.java"]), diff --git a/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/BUILD b/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/BUILD index 173e827dc7..3b04bfbd41 100644 --- a/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/BUILD +++ b/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/BUILD @@ -26,16 +26,13 @@ java_library( java_library( name = "javac", srcs = glob( - [ - "javac/*.java", - ], + ["javac/*.java"], exclude = [ "javac/JavacOptions.java", ], ), deps = [ ":invalid_command_line_exception", - ":javac_options", "//src/java_tools/buildjar/java/com/google/devtools/build/buildjar/javac/plugins", "//third_party:auto_value", "//third_party:guava", diff --git a/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/genclass/BUILD b/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/genclass/BUILD index bd4122b9fc..d85cf60422 100644 --- a/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/genclass/BUILD +++ b/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/genclass/BUILD @@ -1,3 +1,5 @@ +package(default_visibility = [":packages"]) + package_group( name = "packages", packages = [ @@ -5,8 +7,6 @@ package_group( ], ) -package(default_visibility = [":packages"]) - java_binary( name = "GenClass", main_class = "com.google.devtools.build.buildjar.genclass.GenClass", diff --git a/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/jarhelper/BUILD b/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/jarhelper/BUILD index 121cda215d..244e6f4856 100644 --- a/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/jarhelper/BUILD +++ b/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/jarhelper/BUILD @@ -1,3 +1,5 @@ +package(default_visibility = [":packages"]) + package_group( name = "packages", packages = [ @@ -5,8 +7,6 @@ package_group( ], ) -package(default_visibility = [":packages"]) - JARHELPER_SRCS = [ "JarHelper.java", "JarCreator.java", diff --git a/src/java_tools/buildjar/java/com/google/devtools/build/java/turbine/BUILD b/src/java_tools/buildjar/java/com/google/devtools/build/java/turbine/BUILD index 7b8f55ae68..6c109fc22d 100644 --- a/src/java_tools/buildjar/java/com/google/devtools/build/java/turbine/BUILD +++ b/src/java_tools/buildjar/java/com/google/devtools/build/java/turbine/BUILD @@ -1,15 +1,20 @@ +package(default_visibility = [":packages"]) + package_group( name = "packages", packages = ["//src/java_tools/buildjar/..."], ) -package(default_visibility = [":packages"]) - java_binary( name = "turbine", - srcs = ["Turbine.java"], main_class = "com.google.devtools.build.java.turbine.Turbine", visibility = ["//visibility:public"], + runtime_deps = [":turbine_main"], +) + +java_library( + name = "turbine_main", + srcs = ["Turbine.java"], deps = [ "//src/java_tools/buildjar/java/com/google/devtools/build/java/turbine/javac:javac_turbine", "//third_party:guava", @@ -19,6 +24,7 @@ java_binary( filegroup( name = "srcs", - srcs = glob(["**"]) + ["//src/java_tools/buildjar/java/com/google/devtools/build/java/turbine/javac:srcs"], - visibility = ["//src/java_tools/buildjar:__pkg__"], + srcs = glob(["**"]) + [ + "//src/java_tools/buildjar/java/com/google/devtools/build/java/turbine/javac:srcs", + ], ) diff --git a/src/java_tools/buildjar/java/com/google/devtools/build/java/turbine/javac/BUILD b/src/java_tools/buildjar/java/com/google/devtools/build/java/turbine/javac/BUILD index d6c2c90f8d..5cccd6ed1e 100644 --- a/src/java_tools/buildjar/java/com/google/devtools/build/java/turbine/javac/BUILD +++ b/src/java_tools/buildjar/java/com/google/devtools/build/java/turbine/javac/BUILD @@ -1,10 +1,10 @@ +package(default_visibility = [":packages"]) + package_group( name = "packages", packages = ["//src/java_tools/buildjar/..."], ) -package(default_visibility = [":packages"]) - java_library( name = "javac_turbine", srcs = ["JavacTurbine.java"], diff --git a/src/java_tools/buildjar/javatests/com/google/devtools/build/java/bazel/BUILD b/src/java_tools/buildjar/javatests/com/google/devtools/build/java/bazel/BUILD index 4839134bee..9be1d30895 100644 --- a/src/java_tools/buildjar/javatests/com/google/devtools/build/java/bazel/BUILD +++ b/src/java_tools/buildjar/javatests/com/google/devtools/build/java/bazel/BUILD @@ -1,3 +1,7 @@ +# Description: +# Tests for tools for working with Java source code +package(default_visibility = ["//src:__subpackages__"]) + filegroup( name = "srcs", srcs = glob(["**"]), diff --git a/src/java_tools/buildjar/javatests/com/google/devtools/build/java/turbine/BUILD b/src/java_tools/buildjar/javatests/com/google/devtools/build/java/turbine/BUILD index 48379e30d9..6e3b7dd4dc 100644 --- a/src/java_tools/buildjar/javatests/com/google/devtools/build/java/turbine/BUILD +++ b/src/java_tools/buildjar/javatests/com/google/devtools/build/java/turbine/BUILD @@ -1,12 +1,13 @@ -filegroup( - name = "srcs", - srcs = glob(["**"]) + ["//src/java_tools/buildjar/javatests/com/google/devtools/build/java/turbine/javac:srcs"], - visibility = ["//src/java_tools/buildjar:__pkg__"], -) +package(default_visibility = [":packages"]) package_group( name = "packages", packages = ["//src/java_tools/buildjar/..."], ) -package(default_visibility = [":packages"]) +filegroup( + name = "srcs", + srcs = glob(["**"]) + [ + "//src/java_tools/buildjar/javatests/com/google/devtools/build/java/turbine/javac:srcs", + ], +) diff --git a/src/java_tools/buildjar/javatests/com/google/devtools/build/java/turbine/javac/BUILD b/src/java_tools/buildjar/javatests/com/google/devtools/build/java/turbine/javac/BUILD index 233aa117b9..b2c304c2ec 100644 --- a/src/java_tools/buildjar/javatests/com/google/devtools/build/java/turbine/javac/BUILD +++ b/src/java_tools/buildjar/javatests/com/google/devtools/build/java/turbine/javac/BUILD @@ -16,7 +16,6 @@ java_test( ], tags = ["jdk8"], deps = [ - "//src/java_tools/buildjar/java/com/google/devtools/build/buildjar:JarOwner", "//src/java_tools/buildjar/java/com/google/devtools/build/java/turbine/javac:javac_turbine", "//src/main/protobuf:deps_java_proto", "//third_party:asm", diff --git a/src/java_tools/junitrunner/javatests/com/google/testing/junit/runner/BUILD b/src/java_tools/junitrunner/javatests/com/google/testing/junit/runner/BUILD index 95327df8ab..7642858db5 100644 --- a/src/java_tools/junitrunner/javatests/com/google/testing/junit/runner/BUILD +++ b/src/java_tools/junitrunner/javatests/com/google/testing/junit/runner/BUILD @@ -2,8 +2,6 @@ DEFAULT_VISIBILITY = [ "//src/java_tools/junitrunner:__subpackages__", ] -licenses(["notice"]) # Apache 2.0 - # Libraries # ========================================================= java_library( @@ -46,6 +44,7 @@ java_binary( # Tests # ========================================================= + java_test( name = "AllTests", args = glob(["**/*Test.java"]), diff --git a/src/java_tools/junitrunner/javatests/com/google/testing/junit/runner/testbed/BUILD b/src/java_tools/junitrunner/javatests/com/google/testing/junit/runner/testbed/BUILD index 3061e6da64..d48306f2af 100644 --- a/src/java_tools/junitrunner/javatests/com/google/testing/junit/runner/testbed/BUILD +++ b/src/java_tools/junitrunner/javatests/com/google/testing/junit/runner/testbed/BUILD @@ -4,8 +4,6 @@ DEFAULT_VISIBILITY = [ package(default_visibility = DEFAULT_VISIBILITY) -licenses(["notice"]) # Apache 2.0 - java_library( name = "testbed", testonly = 1, diff --git a/src/java_tools/singlejar/BUILD b/src/java_tools/singlejar/BUILD index 24c9ff50c1..f1bd34146a 100644 --- a/src/java_tools/singlejar/BUILD +++ b/src/java_tools/singlejar/BUILD @@ -1,3 +1,6 @@ +# Description: +# SingleJar combines multiple zip files and additional files +# into a single zip file. package(default_visibility = ["//src:__subpackages__"]) filegroup( -- cgit v1.2.3