diff options
author | Damien Martin-Guillerez <dmarting@google.com> | 2016-10-19 13:49:54 +0000 |
---|---|---|
committer | Philipp Wollermann <philwo@google.com> | 2016-10-19 15:01:54 +0000 |
commit | 5f026f96118e76c8a6357c61e5710e0bd1bf0f54 (patch) | |
tree | 64e38f345453d1f9f966a82caa30f2911ddcec12 /src/main/java/com/google/devtools/build/lib/sandbox | |
parent | 212729a59e02f39ae3a1dce2d53dd2ce13c75f92 (diff) |
Rollback of commit e6191f7209ebf3f67480bc7aa9db5ac58beb2156.
*** Reason for rollback ***
Roll-forward with fix.
Tested with `bazel build src:srcs //src/test/...` using a bootstrapped bazel.
Fixes #1923.
RELNOTES[INC]: Non test-only targets can no longer depends on test-only targets.
*** Original change description ***
Automated [] rollback of commit a9f20b0d6459d395444c45cf5e94a899f3443633.
*** Reason for rollback ***
Broke Bazel CI:
ERROR: /home/ci/workspace/Bazel/JAVA_VERSION/1.8/PLATFORM_NAME/linux-x86_64/src/BUILD:284:1: in filegroup rule //src:srcs: non-test target '//src:srcs' depends on testonly target '//src/java_tools/junitrunner/java/com/google/testing/junit/junit4:srcs' and doesn't have testonly attribute set.
ERROR: Analysis of target '//scripts/packages:packages' failed; build aborted.
http://ci.bazel.io/view/Bazel%20b...
***
--
MOS_MIGRATED_REVID=136589937
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/sandbox')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/sandbox/BUILD | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/sandbox/BUILD b/src/main/java/com/google/devtools/build/lib/sandbox/BUILD index 6071bfc5cd..593aafa73e 100644 --- a/src/main/java/com/google/devtools/build/lib/sandbox/BUILD +++ b/src/main/java/com/google/devtools/build/lib/sandbox/BUILD @@ -29,5 +29,6 @@ java_library( filegroup( name = "srcs", + testonly = 0, # All srcs should be not test only, overwrite package default. srcs = glob(["**"]), ) |