Skip to content

Commit

Permalink
fix usage to not expose underlying yaml (ros2#630)
Browse files Browse the repository at this point in the history
Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
  • Loading branch information
dirk-thomas committed Apr 22, 2020
1 parent 0a48e6b commit 3d1337d
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions rcl_yaml_param_parser/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ project(rcl_yaml_param_parser)
find_package(ament_cmake_ros REQUIRED)
find_package(rcutils REQUIRED)
find_package(libyaml_vendor REQUIRED)
find_package(yaml REQUIRED)

# Default to C++14
if(NOT CMAKE_CXX_STANDARD)
Expand All @@ -26,7 +25,7 @@ add_library(
target_include_directories(${PROJECT_NAME} PUBLIC
"$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>"
"$<INSTALL_INTERFACE:include>")
ament_target_dependencies(${PROJECT_NAME} "yaml" "rcutils")
ament_target_dependencies(${PROJECT_NAME} "libyaml_vendor" "rcutils")

# Set the visibility to hidden by default if possible
if(CMAKE_C_COMPILER_ID STREQUAL "GNU" OR CMAKE_C_COMPILER_ID MATCHES "Clang")
Expand Down Expand Up @@ -66,7 +65,7 @@ if(BUILD_TESTING)

endif()

ament_export_dependencies(ament_cmake)
ament_export_dependencies(ament_cmake libyaml_vendor)
ament_export_include_directories(include)
install(
DIRECTORY include/
Expand Down

0 comments on commit 3d1337d

Please sign in to comment.