commit fc5ff3ba083808ce7fe5977aaca3d01d9efdb88f
author Damien Martin-Guillerez <dmarting@google.com> Thu Oct 20 11:52:13 2016 +0200
committer Paul Wankadia <junyer@google.com> Mon Oct 24 13:23:04 2016 +0000
tree 2b7433c5d87fd7eab63b039455ed54abcb3b3c99
parent b94b7cd42e9f02673cd748c1ac1d16db4052514c

Correctly set testonly to test targets

testonly was not enforced in previous version of Bazel but this will
be fixed in the next version of Bazel. Correct it for re2.

Tested with `bazel build ...`

Tracking bug: bazelbuild/bazel#1967

Change-Id: I880326e99bc8ed4904473dcf0939bd7ab3b41b26
Reviewed-on: https://code-review.googlesource.com/8610
Reviewed-by: John Cater <jcater@google.com>
Reviewed-by: Paul Wankadia <junyer@google.com>

BUILD

diff --git a/BUILD b/BUILD
index f4abb0a..5223f6e 100644
--- a/BUILD
+++ b/BUILD
@@ -89,6 +89,7 @@
 
 cc_library(
     name = "test",
+    testonly = 1,
     srcs = ["util/test.cc"],
     deps = [":testing"],
 )
@@ -155,12 +156,14 @@
 
 cc_library(
     name = "benchmark",
+    testonly = 1,
     srcs = ["util/benchmark.cc"],
     deps = [":testing"],
 )
 
 cc_binary(
     name = "regexp_benchmark",
+    testonly = 1,
     srcs = ["re2/testing/regexp_benchmark.cc"],
     linkopts = [
         "-lm",