diff options
author | 2016-10-19 09:56:43 +0000 | |
---|---|---|
committer | 2016-10-19 10:53:07 +0000 | |
commit | e6191f7209ebf3f67480bc7aa9db5ac58beb2156 (patch) | |
tree | 2cfbec84aeb326362061af66e6beba7f578ab9e5 /src/main/java/com/google/devtools/build/docgen/templates/attributes | |
parent | 47fc5b3406c6106b3025534af18c122d7321b861 (diff) |
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%20bootstrap%20and%20maintenance/job/Bazel/JAVA_VERSION=1.8,PLATFORM_NAME=linux-x86_64/923/console
*** Original change description ***
Enforce that non test-only targets cannot depends on test-only targets
This behavior was not enforced since we open-sourced Bazel even though this was
documented and the attributed existed.
Also removed the incorrect part from the documentations: we do not enforce
restrictions on javatests package.
Fixes #1923.
RELNOTES[INC]: Non test-only targets can no longer depends on test-only targets.
--
MOS_MIGRATED_REVID=136574806
Diffstat (limited to 'src/main/java/com/google/devtools/build/docgen/templates/attributes')
-rw-r--r-- | src/main/java/com/google/devtools/build/docgen/templates/attributes/common/testonly.html | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/src/main/java/com/google/devtools/build/docgen/templates/attributes/common/testonly.html b/src/main/java/com/google/devtools/build/docgen/templates/attributes/common/testonly.html index 8d3efe9b94..ed26e6198f 100644 --- a/src/main/java/com/google/devtools/build/docgen/templates/attributes/common/testonly.html +++ b/src/main/java/com/google/devtools/build/docgen/templates/attributes/common/testonly.html @@ -16,6 +16,12 @@ are <code>testonly</code> by default. </p> <p> +By virtue of +<a href="${link package.default_testonly}"><code>default_testonly</code></a>, +targets under <code>javatests</code> are <code>testonly</code> by default. +</p> + +<p> This attribute is intended to mean that the target should not be contained in binaries that are released to production. </p> |