Merge pull request RobotLocomotion#6335 from fbudin69500/Remove_yaml-…

…cpp_installed_extra_headers Only install public headers for yaml_cpp
trigrass2 · Jun 14, 2017 · 74a591b · 74a591b
2 parents 4a7a456 + 2ba5767
commit 74a591b
Showing 1 changed file with 12 additions and 5 deletions.
diff --git a/tools/yaml_cpp.BUILD b/tools/yaml_cpp.BUILD
@@ -1,18 +1,25 @@
 # -*- python -*-
 
-load("@drake//tools:install.bzl", "cmake_config", "install", "install_cmake_config")
+load(
+    "@drake//tools:install.bzl",
+    "cmake_config",
+    "install",
+    "install_cmake_config",
+)
 
 package(default_visibility = ["//visibility:public"])
 
+public_headers = glob([
+    "include/**/*.h",
+])
+
 cc_library(
     name = "yaml_cpp",
     srcs = glob([
         "src/**/*.cpp",
         "src/**/*.h",
     ]),
-    hdrs = glob([
-        "include/**/*.h",
-    ]),
+    hdrs = public_headers,
     includes = ["include"],
 )
 
@@ -26,8 +33,8 @@ install_cmake_config(package = "yaml-cpp")  # Creates rule :install_cmake_config
 
 install(
     name = "install",
+    hdrs = public_headers,
     doc_dest = "share/doc/yaml-cpp",
-    guess_hdrs = "PACKAGE",
     hdr_dest = "include",
     hdr_strip_prefix = ["include"],
     license_docs = ["LICENSE"],