From 7d265e07e7a1e37f04d53342710e4f21d9ee8083 Mon Sep 17 00:00:00 2001 From: Damien Martin-Guillerez Date: Fri, 1 Jul 2016 13:33:48 +0000 Subject: Add all the sources to //:srcs filegroup and add a check to detect missing file to it. We need to activate this check on presubmits -- Change-Id: Ia95e92d3816ce92bb69bc0e2cf56e9c60b68d970 Reviewed-on: https://bazel-review.googlesource.com/#/c/3949/ MOS_MIGRATED_REVID=126404792 --- BUILD | 1 + compile.sh | 34 +++++++++++++++++++++- examples/BUILD | 4 ++- examples/android/java/bazel/BUILD | 6 ++++ examples/cpp/BUILD | 5 +--- examples/j2objc/BUILD | 6 +--- examples/java-native/BUILD | 2 +- examples/java-skylark/BUILD | 2 +- examples/shell/BUILD | 2 +- scripts/BUILD | 5 +++- scripts/packages/BUILD | 6 ++++ scripts/release/BUILD | 6 ++++ src/BUILD | 10 ++++++- src/java_tools/buildjar/BUILD | 2 ++ .../com/google/devtools/build/java/bazel/BUILD | 6 ++++ .../com/google/devtools/build/java/turbine/BUILD | 6 ++++ .../google/devtools/build/java/turbine/javac/BUILD | 6 ++++ src/test/cpp/BUILD | 6 ++++ src/test/cpp/util/BUILD | 6 ++++ .../java/com/google/devtools/build/android/BUILD | 11 +++++++ .../google/devtools/build/android/ideinfo/BUILD | 6 ++++ .../google/devtools/build/android/idlclass/BUILD | 6 ++++ .../google/devtools/build/android/resources/BUILD | 6 ++++ .../google/devtools/build/android/ziputils/BUILD | 6 ++++ .../java/com/google/devtools/build/docgen/BUILD | 6 ++++ src/test/java/com/google/devtools/build/lib/BUILD | 12 ++++++++ .../devtools/build/lib/bazel/repository/BUILD | 6 ++++ .../build/lib/bazel/repository/downloader/BUILD | 6 ++++ .../com/google/devtools/build/lib/buildtool/BUILD | 6 ++++ .../devtools/build/lib/rules/repository/BUILD | 6 ++++ .../com/google/devtools/build/lib/skyframe/BUILD | 6 ++++ .../com/google/devtools/build/lib/skylark/BUILD | 6 ++++ .../java/com/google/devtools/build/skyframe/BUILD | 6 ++++ .../java/com/google/devtools/common/options/BUILD | 6 ++++ src/test/shell/BUILD | 9 ++++++ src/test/shell/bazel/BUILD | 11 ++++++- src/test/shell/bazel/android/BUILD | 6 ++++ src/test/shell/bazel/testdata/BUILD | 6 ++++ src/test/shell/integration/BUILD | 6 ++++ src/tools/generate_workspace/BUILD | 5 +++- .../java/com/google/devtools/build/workspace/BUILD | 8 +++++ .../google/devtools/build/workspace/maven/BUILD | 6 ++++ src/tools/remote_worker/BUILD | 6 ++++ .../java/com/google/devtools/build/remote/BUILD | 6 ++++ third_party/BUILD | 1 + third_party/java/jdk/langtools/BUILD | 5 ++++ third_party/nanopb/BUILD | 6 ++++ tools/BUILD | 1 + tools/build_defs/apple/test/BUILD | 5 +++- tools/test/BUILD | 5 +--- 50 files changed, 291 insertions(+), 23 deletions(-) diff --git a/BUILD b/BUILD index f7fea2174c..0e72565571 100644 --- a/BUILD +++ b/BUILD @@ -27,6 +27,7 @@ filegroup( "bazel-*", # convenience symlinks "out", # IntelliJ with setup-intellij.sh "output", # output of compile.sh + "WORKSPACE.user.bzl", # generated workspace file ".*", # mainly .git* files ], ) + [ diff --git a/compile.sh b/compile.sh index 59ec71c72e..eceefa5c38 100755 --- a/compile.sh +++ b/compile.sh @@ -39,21 +39,24 @@ function usage() { echo " Commands for developers:" >&2 echo " all = compile,determinism,test" >&2 echo " determinism = test for stability of Bazel builds" >&2 + echo " srcs = test that //:srcs contains all the sources" >&2 echo " test = run the full test suite of Bazel" >&2 exit 1 } function parse_options() { - local keywords="(compile|all|determinism|bootstrap|test)" + local keywords="(compile|all|determinism|bootstrap|srcs|test)" COMMANDS="${1:-compile}" [[ "${COMMANDS}" =~ ^$keywords(,$keywords)*$ ]] || usage "$@" DO_COMPILE= DO_CHECKSUM= DO_FULL_CHECKSUM=1 DO_TESTS= + DO_SRCS_TEST= [[ "${COMMANDS}" =~ (compile|all) ]] && DO_COMPILE=1 [[ "${COMMANDS}" =~ (bootstrap|determinism|all) ]] && DO_CHECKSUM=1 [[ "${COMMANDS}" =~ (bootstrap) ]] && DO_FULL_CHECKSUM= + [[ "${COMMANDS}" =~ (srcs|all) ]] && DO_SRCS_TEST=1 [[ "${COMMANDS}" =~ (test|all) ]] && DO_TESTS=1 BAZEL_BIN=${2:-"bazel-bin/src/bazel"} @@ -127,6 +130,35 @@ if [ $DO_CHECKSUM ]; then fi fi +# +# Test that //:srcs contains all the sources +# +if [ $DO_SRCS_TEST ]; then + new_step "Checking that //:srcs contains all the sources" + log "Querying //:srcs" + ${BAZEL} query 'kind("source file", deps(//:srcs))' 2>/dev/null \ + | grep -v '^@' \ + | sed -e 's|^//||' | sed 's|^:||' | sed 's|:|/|' \ + | sort -u >"${OUTPUT_DIR}/srcs-query" + + log "Finding all files" + # See file BUILD for the list of grep -v exceptions. + # tools/defaults package is hidden by Bazel so cannot be put in the srcs. + find . -type f | sed 's|./||' \ + | grep -v '^bazel-' | grep -v '^WORKSPACE.user.bzl' \ + | grep -v '^\.' | grep -v '^out/' | grep -v '^output/' \ + | grep -v '^tools/defaults/BUILD' \ + | sort -u >"${OUTPUT_DIR}/srcs-find" + + log "Diffing" + res="$(diff -U 0 "${OUTPUT_DIR}/srcs-find" "${OUTPUT_DIR}/srcs-query" | sed 's|^-||' | grep -Ev '^(@@|\+\+|--)' || true)" + + if [ -n "${res}" ]; then + fail "//:srcs filegroup do not contains all the sources, missing: +${res}" + fi +fi + # # Tests # diff --git a/examples/BUILD b/examples/BUILD index 781c81a46c..b2ec2848e8 100644 --- a/examples/BUILD +++ b/examples/BUILD @@ -2,11 +2,13 @@ package(default_visibility = ["//visibility:public"]) filegroup( name = "srcs", - srcs = [ + srcs = glob(["**"]) + [ + "//examples/android/java/bazel:srcs", "//examples/cpp:srcs", "//examples/gen:srcs", "//examples/java-native:srcs", "//examples/java-skylark:srcs", + "//examples/j2objc:srcs", "//examples/objc:srcs", "//examples/py:srcs", "//examples/py_native:srcs", diff --git a/examples/android/java/bazel/BUILD b/examples/android/java/bazel/BUILD index 293fb5dd45..ad550b5494 100644 --- a/examples/android/java/bazel/BUILD +++ b/examples/android/java/bazel/BUILD @@ -1,3 +1,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//examples:__pkg__"], +) + android_library( name = "lib", srcs = ["Lib.java"], diff --git a/examples/cpp/BUILD b/examples/cpp/BUILD index 7c0543e1f4..3048a0d48e 100644 --- a/examples/cpp/BUILD +++ b/examples/cpp/BUILD @@ -26,8 +26,5 @@ cc_test( filegroup( name = "srcs", - srcs = ["BUILD"] + glob([ - "**/*.cc", - "**/*.h", - ]), + srcs = glob(["**"]), ) diff --git a/examples/j2objc/BUILD b/examples/j2objc/BUILD index 41c7bb740a..0a3d3894cf 100644 --- a/examples/j2objc/BUILD +++ b/examples/j2objc/BUILD @@ -42,10 +42,6 @@ objc_binary( filegroup( name = "srcs", - srcs = glob([ - "J2ObjcExample/*", - "src/**", - "BUILD", - ]), + srcs = glob(["**"]), visibility = ["//examples:__pkg__"], ) diff --git a/examples/java-native/BUILD b/examples/java-native/BUILD index b698a88145..4f4ecd107d 100644 --- a/examples/java-native/BUILD +++ b/examples/java-native/BUILD @@ -2,7 +2,7 @@ package(default_visibility = ["//visibility:public"]) filegroup( name = "srcs", - srcs = [ + srcs = glob(["**"]) + [ "//examples/java-native/src/main/java/com/example/myproject:srcs", "//examples/java-native/src/main/resources:srcs", "//examples/java-native/src/test/java/com/example/myproject:srcs", diff --git a/examples/java-skylark/BUILD b/examples/java-skylark/BUILD index 9a76f2b0e9..74541aede1 100644 --- a/examples/java-skylark/BUILD +++ b/examples/java-skylark/BUILD @@ -2,7 +2,7 @@ package(default_visibility = ["//visibility:public"]) filegroup( name = "srcs", - srcs = [ + srcs = glob(["**"]) + [ "//examples/java-skylark/src/main/java/com/example/myproject:srcs", "//examples/java-skylark/src/main/resources:srcs", "//examples/java-skylark/src/test/java/com/example/myproject:srcs", diff --git a/examples/shell/BUILD b/examples/shell/BUILD index 9aba320bc2..4b51df1cb3 100644 --- a/examples/shell/BUILD +++ b/examples/shell/BUILD @@ -24,5 +24,5 @@ sh_test( filegroup( name = "srcs", - srcs = ["BUILD"] + glob(["**/*.sh"]) + glob(["**/*.txt"]), + srcs = glob(["**"]), ) diff --git a/scripts/BUILD b/scripts/BUILD index 26aab55491..398ce4ca48 100644 --- a/scripts/BUILD +++ b/scripts/BUILD @@ -34,6 +34,9 @@ sh_test( filegroup( name = "srcs", - srcs = glob(["**"]), + srcs = glob(["**"]) + [ + "//scripts/release:srcs", + "//scripts/packages:srcs", + ], visibility = ["//:__pkg__"], ) diff --git a/scripts/packages/BUILD b/scripts/packages/BUILD index 05702d12b9..44b4e1c736 100644 --- a/scripts/packages/BUILD +++ b/scripts/packages/BUILD @@ -1,3 +1,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//scripts:__pkg__"], +) + sh_binary( name = "package-info-generator", srcs = ["package_info_generator.sh"], diff --git a/scripts/release/BUILD b/scripts/release/BUILD index 7ccd9d5596..a6a4ab0801 100644 --- a/scripts/release/BUILD +++ b/scripts/release/BUILD @@ -1,6 +1,12 @@ # Scripts for building Bazel releases package(default_visibility = ["//visibility:private"]) +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//scripts:__pkg__"], +) + sh_library( name = "relnotes", srcs = ["relnotes.sh"], diff --git a/src/BUILD b/src/BUILD index 13538cdced..321f7fe736 100644 --- a/src/BUILD +++ b/src/BUILD @@ -133,7 +133,7 @@ genrule( "//third_party:gpl-srcs", "//third_party/iossim:srcs", "//third_party/java/jarjar:srcs", - "//third_party/java/jdk/langtools:srcs", + "//third_party/java/jdk/langtools:test-srcs", "//third_party/py/concurrent:srcs", "//third_party/py/gflags:srcs", "//src/java_tools/buildjar/java/com/google/devtools/build/buildjar/jarhelper:srcs", @@ -295,6 +295,13 @@ filegroup( "//src/objc_tools/bundlemerge:srcs", "//src/objc_tools/plmerge:srcs", "//src/objc_tools/xcodegen:srcs", + "//src/test/cpp:srcs", + "//src/test/java/com/google/devtools/build/android:srcs", + "//src/test/java/com/google/devtools/build/docgen:srcs", + "//src/test/java/com/google/devtools/build/lib:srcs", + "//src/test/java/com/google/devtools/build/skyframe:srcs", + "//src/test/java/com/google/devtools/common/options:srcs", + "//src/test/shell:srcs", "//src/tools/android/java/com/google/devtools/build/android:srcs", "//src/tools/generate_workspace:srcs", "//src/tools/xcode/actoolwrapper:srcs", @@ -309,6 +316,7 @@ filegroup( "//src/tools/xcode/xcodelocator:srcs", "//src/tools/xcode/xcrunwrapper:srcs", "//src/tools/xcode-common:srcs", + "//src/tools/remote_worker:srcs", ], visibility = ["//:__pkg__"], ) diff --git a/src/java_tools/buildjar/BUILD b/src/java_tools/buildjar/BUILD index 56298dfd1a..5c379c06d1 100644 --- a/src/java_tools/buildjar/BUILD +++ b/src/java_tools/buildjar/BUILD @@ -41,6 +41,8 @@ filegroup( "//src/java_tools/buildjar/java/com/google/devtools/build/buildjar:srcs", "//src/java_tools/buildjar/java/com/google/devtools/build/java/bazel:srcs", "//src/java_tools/buildjar/java/com/google/devtools/build/java/turbine:srcs", + "//src/java_tools/buildjar/javatests/com/google/devtools/build/java/bazel:srcs", + "//src/java_tools/buildjar/javatests/com/google/devtools/build/java/turbine:srcs", ], visibility = ["//src:__pkg__"], ) 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 324082c230..4839134bee 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,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//src/java_tools/buildjar:__pkg__"], +) + java_test( name = "BazelJavaCompilerTest", size = "small", 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 f454578343..317b03b2ae 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,3 +1,9 @@ +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_group( name = "packages", packages = ["//src/java_tools/buildjar/..."], 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 4132bba9df..2127783f7f 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 @@ -1,3 +1,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//src/java_tools/buildjar/javatests/com/google/devtools/build/java/turbine:__pkg__"], +) + java_test( name = "JavacTurbineTest", srcs = ["JavacTurbineTest.java"], diff --git a/src/test/cpp/BUILD b/src/test/cpp/BUILD index 892d73b99c..acc5461aaf 100644 --- a/src/test/cpp/BUILD +++ b/src/test/cpp/BUILD @@ -2,6 +2,12 @@ # C++ utility tests for Bazel package(default_visibility = ["//visibility:public"]) +filegroup( + name = "srcs", + srcs = glob(["**"]) + ["//src/test/cpp/util:srcs"], + visibility = ["//src:__pkg__"], +) + cc_test( name = "blaze_util_test", srcs = ["blaze_util_test.cc"], diff --git a/src/test/cpp/util/BUILD b/src/test/cpp/util/BUILD index c9a472013e..0ec570063e 100644 --- a/src/test/cpp/util/BUILD +++ b/src/test/cpp/util/BUILD @@ -2,6 +2,12 @@ # C++ utility tests for Bazel package(default_visibility = ["//visibility:public"]) +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//src/test/cpp:__pkg__"], +) + cc_test( name = "md5_test", srcs = ["md5_test.cc"], diff --git a/src/test/java/com/google/devtools/build/android/BUILD b/src/test/java/com/google/devtools/build/android/BUILD index fd1ad295f7..bf9db4ab74 100644 --- a/src/test/java/com/google/devtools/build/android/BUILD +++ b/src/test/java/com/google/devtools/build/android/BUILD @@ -1,3 +1,14 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]) + [ + "//src/test/java/com/google/devtools/build/android/ideinfo:srcs", + "//src/test/java/com/google/devtools/build/android/idlclass:srcs", + "//src/test/java/com/google/devtools/build/android/resources:srcs", + "//src/test/java/com/google/devtools/build/android/ziputils:srcs", + ], + visibility = ["//src:__pkg__"], +) + java_test( name = "AndroidResourceCompilationActionTest", size = "medium", diff --git a/src/test/java/com/google/devtools/build/android/ideinfo/BUILD b/src/test/java/com/google/devtools/build/android/ideinfo/BUILD index 4bf367e35d..9de5aa76d4 100644 --- a/src/test/java/com/google/devtools/build/android/ideinfo/BUILD +++ b/src/test/java/com/google/devtools/build/android/ideinfo/BUILD @@ -1,3 +1,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//src/test/java/com/google/devtools/build/android:__pkg__"], +) + java_test( name = "PackageParserTest", size = "small", diff --git a/src/test/java/com/google/devtools/build/android/idlclass/BUILD b/src/test/java/com/google/devtools/build/android/idlclass/BUILD index d06e046eeb..46f2159177 100644 --- a/src/test/java/com/google/devtools/build/android/idlclass/BUILD +++ b/src/test/java/com/google/devtools/build/android/idlclass/BUILD @@ -1,3 +1,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//src/test/java/com/google/devtools/build/android:__pkg__"], +) + java_test( name = "IdlClassTest", size = "medium", diff --git a/src/test/java/com/google/devtools/build/android/resources/BUILD b/src/test/java/com/google/devtools/build/android/resources/BUILD index 5ea3ac5d6b..fa8ef74abf 100644 --- a/src/test/java/com/google/devtools/build/android/resources/BUILD +++ b/src/test/java/com/google/devtools/build/android/resources/BUILD @@ -1,3 +1,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//src/test/java/com/google/devtools/build/android:__pkg__"], +) + java_test( name = "RClassWriterTest", size = "medium", diff --git a/src/test/java/com/google/devtools/build/android/ziputils/BUILD b/src/test/java/com/google/devtools/build/android/ziputils/BUILD index 643e29eeba..9507c5a290 100644 --- a/src/test/java/com/google/devtools/build/android/ziputils/BUILD +++ b/src/test/java/com/google/devtools/build/android/ziputils/BUILD @@ -1,3 +1,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//src/test/java/com/google/devtools/build/android:__pkg__"], +) + java_test( name = "ziputils-tests", srcs = glob(["*.java"]), diff --git a/src/test/java/com/google/devtools/build/docgen/BUILD b/src/test/java/com/google/devtools/build/docgen/BUILD index 5376077b12..6c21717373 100644 --- a/src/test/java/com/google/devtools/build/docgen/BUILD +++ b/src/test/java/com/google/devtools/build/docgen/BUILD @@ -4,6 +4,12 @@ package( licenses(["notice"]) # Apache 2.0 +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//src:__pkg__"], +) + test_suite( name = "all_tests", tags = ["docgen"], diff --git a/src/test/java/com/google/devtools/build/lib/BUILD b/src/test/java/com/google/devtools/build/lib/BUILD index 45763d0c69..5f997d5b0c 100644 --- a/src/test/java/com/google/devtools/build/lib/BUILD +++ b/src/test/java/com/google/devtools/build/lib/BUILD @@ -2,6 +2,18 @@ package( default_visibility = ["//src/test/java/com/google/devtools/build/lib:__subpackages__"], ) +filegroup( + name = "srcs", + srcs = glob(["**"]) + [ + "//src/test/java/com/google/devtools/build/lib/skylark:srcs", + "//src/test/java/com/google/devtools/build/lib/skyframe:srcs", + "//src/test/java/com/google/devtools/build/lib/rules/repository:srcs", + "//src/test/java/com/google/devtools/build/lib/bazel/repository:srcs", + "//src/test/java/com/google/devtools/build/lib/buildtool:srcs", + ], + visibility = ["//src:__pkg__"], +) + # This should correspond to the list of "EMBEDDED_TOOLS" in TestConstants.java.bazel. filegroup( name = "embedded_scripts", diff --git a/src/test/java/com/google/devtools/build/lib/bazel/repository/BUILD b/src/test/java/com/google/devtools/build/lib/bazel/repository/BUILD index 6ee8c565fa..0121f7be79 100644 --- a/src/test/java/com/google/devtools/build/lib/bazel/repository/BUILD +++ b/src/test/java/com/google/devtools/build/lib/bazel/repository/BUILD @@ -1,3 +1,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]) + ["//src/test/java/com/google/devtools/build/lib/bazel/repository/downloader:srcs"], + visibility = ["//src/test/java/com/google/devtools/build/lib:__pkg__"], +) + java_test( name = "RepositoryTests", srcs = glob([ diff --git a/src/test/java/com/google/devtools/build/lib/bazel/repository/downloader/BUILD b/src/test/java/com/google/devtools/build/lib/bazel/repository/downloader/BUILD index 308bdec243..f29ceb4e7a 100644 --- a/src/test/java/com/google/devtools/build/lib/bazel/repository/downloader/BUILD +++ b/src/test/java/com/google/devtools/build/lib/bazel/repository/downloader/BUILD @@ -1,3 +1,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//src/test/java/com/google/devtools/build/lib/bazel/repository:__pkg__"], +) + java_test( name = "DownloaderTests", srcs = glob(["*.java"]), diff --git a/src/test/java/com/google/devtools/build/lib/buildtool/BUILD b/src/test/java/com/google/devtools/build/lib/buildtool/BUILD index dc1559e4ce..d18063a5ee 100644 --- a/src/test/java/com/google/devtools/build/lib/buildtool/BUILD +++ b/src/test/java/com/google/devtools/build/lib/buildtool/BUILD @@ -1,3 +1,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//src/test/java/com/google/devtools/build/lib:__pkg__"], +) + java_library( name = "testutil", srcs = glob(["util/*.java"]), diff --git a/src/test/java/com/google/devtools/build/lib/rules/repository/BUILD b/src/test/java/com/google/devtools/build/lib/rules/repository/BUILD index 6e16c4f46e..9058c97216 100644 --- a/src/test/java/com/google/devtools/build/lib/rules/repository/BUILD +++ b/src/test/java/com/google/devtools/build/lib/rules/repository/BUILD @@ -1,3 +1,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//src/test/java/com/google/devtools/build/lib:__pkg__"], +) + java_test( name = "RepositoryTests", srcs = glob(["*.java"]), diff --git a/src/test/java/com/google/devtools/build/lib/skyframe/BUILD b/src/test/java/com/google/devtools/build/lib/skyframe/BUILD index bd2f417b75..9d81329fe1 100644 --- a/src/test/java/com/google/devtools/build/lib/skyframe/BUILD +++ b/src/test/java/com/google/devtools/build/lib/skyframe/BUILD @@ -1,3 +1,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//src/test/java/com/google/devtools/build/lib:__pkg__"], +) + java_library( name = "testutil", srcs = glob([ diff --git a/src/test/java/com/google/devtools/build/lib/skylark/BUILD b/src/test/java/com/google/devtools/build/lib/skylark/BUILD index 2580fb27c3..c81772a4b6 100644 --- a/src/test/java/com/google/devtools/build/lib/skylark/BUILD +++ b/src/test/java/com/google/devtools/build/lib/skylark/BUILD @@ -1,3 +1,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//src/test/java/com/google/devtools/build/lib:__pkg__"], +) + java_library( name = "testutil", srcs = glob([ diff --git a/src/test/java/com/google/devtools/build/skyframe/BUILD b/src/test/java/com/google/devtools/build/skyframe/BUILD index fccede9c55..21b63420ea 100644 --- a/src/test/java/com/google/devtools/build/skyframe/BUILD +++ b/src/test/java/com/google/devtools/build/skyframe/BUILD @@ -1,3 +1,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//src:__pkg__"], +) + TESTUTIL_FILES = [ "TrackingAwaiter.java", "GraphTester.java", diff --git a/src/test/java/com/google/devtools/common/options/BUILD b/src/test/java/com/google/devtools/common/options/BUILD index 2cd7862068..d11c881e5b 100644 --- a/src/test/java/com/google/devtools/common/options/BUILD +++ b/src/test/java/com/google/devtools/common/options/BUILD @@ -1,3 +1,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//src:__pkg__"], +) + java_test( name = "options_test", srcs = glob(["*.java"]), diff --git a/src/test/shell/BUILD b/src/test/shell/BUILD index 4055e4469b..95c387b055 100644 --- a/src/test/shell/BUILD +++ b/src/test/shell/BUILD @@ -1,5 +1,14 @@ package(default_visibility = ["//visibility:private"]) +filegroup( + name = "srcs", + srcs = glob(["**"]) + [ + "//src/test/shell/bazel:srcs", + "//src/test/shell/integration:srcs", + ], + visibility = ["//src:__pkg__"], +) + sh_library( name = "bashunit", srcs = ["unittest.bash"], diff --git a/src/test/shell/bazel/BUILD b/src/test/shell/bazel/BUILD index 40b34bc359..e064d72f87 100644 --- a/src/test/shell/bazel/BUILD +++ b/src/test/shell/bazel/BUILD @@ -1,5 +1,14 @@ package(default_visibility = ["//visibility:private"]) +filegroup( + name = "srcs", + srcs = glob(["**"]) + [ + "//src/test/shell/bazel/android:srcs", + "//src/test/shell/bazel/testdata:srcs", + ], + visibility = ["//src/test/shell:__pkg__"], +) + genrule( name = "langtools-copy", testonly = 1, @@ -60,7 +69,7 @@ filegroup( "//src/test/shell:bashunit", "//third_party:srcs", "//third_party/ijar", - "//third_party/java/jdk/langtools:srcs", + "//third_party/java/jdk/langtools:test-srcs", "//tools:srcs", ], visibility = [ diff --git a/src/test/shell/bazel/android/BUILD b/src/test/shell/bazel/android/BUILD index 16243889ff..26c155a568 100644 --- a/src/test/shell/bazel/android/BUILD +++ b/src/test/shell/bazel/android/BUILD @@ -1,3 +1,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//src/test/shell/bazel:__pkg__"], +) + sh_test( name = "android_integration_test", size = "large", diff --git a/src/test/shell/bazel/testdata/BUILD b/src/test/shell/bazel/testdata/BUILD index 6064737e6d..c3e91a5e31 100644 --- a/src/test/shell/bazel/testdata/BUILD +++ b/src/test/shell/bazel/testdata/BUILD @@ -1,3 +1,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//src/test/shell/bazel:__pkg__"], +) + filegroup( name = "git-repos", testonly = 1, diff --git a/src/test/shell/integration/BUILD b/src/test/shell/integration/BUILD index 768f19096d..35958d4ce7 100644 --- a/src/test/shell/integration/BUILD +++ b/src/test/shell/integration/BUILD @@ -1,5 +1,11 @@ package(default_visibility = ["//visibility:private"]) +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//src/test/shell:__pkg__"], +) + filegroup( name = "test-deps", testonly = 1, diff --git a/src/tools/generate_workspace/BUILD b/src/tools/generate_workspace/BUILD index 5735e7328a..96bba5c293 100644 --- a/src/tools/generate_workspace/BUILD +++ b/src/tools/generate_workspace/BUILD @@ -9,6 +9,9 @@ java_binary( filegroup( name = "srcs", - srcs = glob(["**"]) + ["//src/tools/generate_workspace/src/main/java/com/google/devtools/build/workspace:srcs"], + srcs = glob(["**"]) + [ + "//src/tools/generate_workspace/src/main/java/com/google/devtools/build/workspace:srcs", + "//src/tools/generate_workspace/src/test/java/com/google/devtools/build/workspace:srcs", + ], visibility = ["//src:__pkg__"], ) diff --git a/src/tools/generate_workspace/src/test/java/com/google/devtools/build/workspace/BUILD b/src/tools/generate_workspace/src/test/java/com/google/devtools/build/workspace/BUILD index 393a3c04e1..93e5986937 100644 --- a/src/tools/generate_workspace/src/test/java/com/google/devtools/build/workspace/BUILD +++ b/src/tools/generate_workspace/src/test/java/com/google/devtools/build/workspace/BUILD @@ -1,5 +1,13 @@ licenses(["notice"]) # Apache 2.0 +filegroup( + name = "srcs", + srcs = glob(["**"]) + [ + "//src/tools/generate_workspace/src/test/java/com/google/devtools/build/workspace/maven:srcs", + ], + visibility = ["//src/tools/generate_workspace:__pkg__"], +) + java_test( name = "ResultWriterTest", srcs = ["ResultWriterTest.java"], diff --git a/src/tools/generate_workspace/src/test/java/com/google/devtools/build/workspace/maven/BUILD b/src/tools/generate_workspace/src/test/java/com/google/devtools/build/workspace/maven/BUILD index f02721b560..8db2e7381a 100644 --- a/src/tools/generate_workspace/src/test/java/com/google/devtools/build/workspace/maven/BUILD +++ b/src/tools/generate_workspace/src/test/java/com/google/devtools/build/workspace/maven/BUILD @@ -1,3 +1,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//src/tools/generate_workspace/src/test/java/com/google/devtools/build/workspace:__pkg__"], +) + java_test( name = "RuleTest", srcs = ["RuleTest.java"], diff --git a/src/tools/remote_worker/BUILD b/src/tools/remote_worker/BUILD index 7f44a71e83..b8ed480ae1 100644 --- a/src/tools/remote_worker/BUILD +++ b/src/tools/remote_worker/BUILD @@ -1,3 +1,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]) + ["//src/tools/remote_worker/src/main/java/com/google/devtools/build/remote:srcs"], + visibility = ["//src:__pkg__"], +) + java_binary( name = "remote_worker", main_class = "com.google.devtools.build.remote.RemoteWorker", diff --git a/src/tools/remote_worker/src/main/java/com/google/devtools/build/remote/BUILD b/src/tools/remote_worker/src/main/java/com/google/devtools/build/remote/BUILD index 495d640ba8..d41b79b608 100644 --- a/src/tools/remote_worker/src/main/java/com/google/devtools/build/remote/BUILD +++ b/src/tools/remote_worker/src/main/java/com/google/devtools/build/remote/BUILD @@ -1,3 +1,9 @@ +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//src/tools/remote_worker:__pkg__"], +) + java_library( name = "remote", srcs = glob(["*.java"]), diff --git a/third_party/BUILD b/third_party/BUILD index 750c5ce817..44470af7e8 100644 --- a/third_party/BUILD +++ b/third_party/BUILD @@ -25,6 +25,7 @@ filegroup( "//third_party/py/mock:srcs", "//third_party/py/six:srcs", "//third_party/zlib:srcs", + "//third_party/nanopb:srcs", ], ) diff --git a/third_party/java/jdk/langtools/BUILD b/third_party/java/jdk/langtools/BUILD index e87fa27c1d..5692999306 100644 --- a/third_party/java/jdk/langtools/BUILD +++ b/third_party/java/jdk/langtools/BUILD @@ -4,6 +4,11 @@ licenses(["restricted"]) # GNU GPL v2 with Classpath exception filegroup( name = "srcs", + srcs = glob(["**"]), +) + +filegroup( + name = "test-srcs", srcs = select({ "//tools/jdk:jdk7": [ "BUILD", diff --git a/third_party/nanopb/BUILD b/third_party/nanopb/BUILD index 49f8257be3..e0d16b6049 100644 --- a/third_party/nanopb/BUILD +++ b/third_party/nanopb/BUILD @@ -1,5 +1,11 @@ licenses(["unencumbered"]) +filegroup( + name = "srcs", + srcs = glob(["**"]), + visibility = ["//third_party:__pkg__"], +) + cc_library( name = "nanopb", srcs = glob(["*.c"]), diff --git a/tools/BUILD b/tools/BUILD index 668957498d..360d9eb6b3 100644 --- a/tools/BUILD +++ b/tools/BUILD @@ -11,6 +11,7 @@ filegroup( "//tools/android/jack:srcs", "//tools/buildstamp:srcs", "//tools/build_defs/apple:srcs", + "//tools/build_defs/apple/test:srcs", "//tools/build_defs/docker:srcs", "//tools/build_defs/pkg:srcs", "//tools/build_rules:srcs", diff --git a/tools/build_defs/apple/test/BUILD b/tools/build_defs/apple/test/BUILD index dbc9098fb2..62328aad84 100644 --- a/tools/build_defs/apple/test/BUILD +++ b/tools/build_defs/apple/test/BUILD @@ -1,5 +1,8 @@ filegroup( name = "srcs", srcs = glob(["**"]), - visibility = ["//src/test/shell/bazel:__pkg__"], + visibility = [ + "//src/test/shell/bazel:__pkg__", + "//tools:__pkg__", + ], ) diff --git a/tools/test/BUILD b/tools/test/BUILD index 0631350e53..846016ba8b 100644 --- a/tools/test/BUILD +++ b/tools/test/BUILD @@ -19,8 +19,5 @@ filegroup( filegroup( name = "srcs", - srcs = [ - "BUILD", - "test-setup.sh", - ], + srcs = glob(["*"]), ) -- cgit v1.2.3