diff options
| author |  Thiago Farina <tfarina@chromium.org> | 2016-04-05 13:00:30 +0000 |
|---|---|---|
| committer |  Kristina Chodorow <kchodorow@google.com> | 2016-04-05 14:09:15 +0000 |
| commit | 62d1e72bac93d1cb240630c1ff9777b994712216 (patch) | |
| tree | c33da003476ff2a16eb030889b5f0222f917b6ce | |
| parent | 74d179acce3434a6d0cf7d0488e22472161aca37 (diff) | |
cpp: remove unnecessary visibility attribute from blaze_util target
Commit 5d737d642623: ("Some minor updates to the BUILD files.") made the
default visibility for the whole package public, and thus setting it
again in blaze_util target is rendundat.
While at it remove blaze_util_platform.h from srcs attribute of client
target, as it is already in hdrs list of blaze_util target and client
already depends on it, and thus it is not necessary to list it again
here.
--
Change-Id: I995c3cd6e2f63384acef4b24b9a082e7cb59331a
Reviewed-on: https://bazel-review.googlesource.com/#/c/3150/1
MOS_MIGRATED_REVID=119041634
| -rw-r--r-- | src/main/cpp/BUILD | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/main/cpp/BUILD b/src/main/cpp/BUILD index 2140b181cf..3a5dfc5f36 100644 --- a/src/main/cpp/BUILD +++ b/src/main/cpp/BUILD @@ -44,7 +44,6 @@ cc_library( "-lrt", ], }), - visibility = ["//visibility:public"], deps = [ "//src/main/cpp/util", "//src/main/cpp/util:blaze_exit_code", @@ -72,13 +71,12 @@ cc_binary( "blaze_startup_options.cc", "blaze_startup_options.h", "blaze_startup_options_common.cc", - "blaze_util_platform.h", "option_processor.cc", "option_processor.h", ], copts = [ "-Wno-sign-compare", - "-DBLAZE_JAVA_CPU=\\\"k8\\\"", + "-DBLAZE_JAVA_CPU=\"k8\"", ], linkopts = select({ "//src:darwin": [ |