Skip to content

Commit

Permalink
Explicit yaml dependency and doctext update
Browse files Browse the repository at this point in the history
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
  • Loading branch information
emersonknapp committed Mar 23, 2023
1 parent ad70114 commit 9245a40
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 2 deletions.
3 changes: 3 additions & 0 deletions rcl/CMakeLists.txt
Expand Up @@ -4,6 +4,7 @@ project(rcl)

find_package(ament_cmake_ros REQUIRED)

find_package(libyaml_vendor REQUIRED)
find_package(rcl_interfaces REQUIRED)
find_package(rcl_logging_interface REQUIRED)
find_package(rcl_yaml_param_parser REQUIRED)
Expand All @@ -14,6 +15,7 @@ find_package(rosidl_runtime_c REQUIRED)
find_package(service_msgs REQUIRED)
find_package(tracetools REQUIRED)
find_package(type_description_interfaces REQUIRED)
find_package(yaml REQUIRED)

include(cmake/rcl_set_symbol_visibility_hidden.cmake)
include(cmake/get_default_rcl_logging_implementation.cmake)
Expand Down Expand Up @@ -89,6 +91,7 @@ ament_target_dependencies(${PROJECT_NAME}
"service_msgs"
"tracetools"
"type_description_interfaces"
"yaml"
)

# Causes the visibility macros to use dllexport rather than dllimport,
Expand Down
6 changes: 4 additions & 2 deletions rcl/include/rcl/type_hash.h
Expand Up @@ -43,7 +43,8 @@ extern "C"
* the JSON representation of type_description. Note that output_repr will have a
* terminating null character, which should be omitted from hashing. To do so, use
* (output_repr.buffer_length - 1) or strlen(output_repr.buffer) for the size of data to hash.
* \return RCL_RET_OK on success, RCL_RET_ERROR if any problems occur in translation
* \return #RCL_RET_OK on success, or
* \return #RCL_RET_ERROR if any problems occur in translation.
*/
RCL_PUBLIC
rcl_ret_t
Expand All @@ -61,7 +62,8 @@ rcl_type_description_to_hashable_json(
*
* \param[in] msg Prefilled TypeDescription message describing the type to be hashed
* \param[out] message_digest Preallocated buffer, to be filled with calculated checksum
* \return RCL_RET_OK on success, RCL_RET_ERROR if any problems occur while hashing
* \return #RCL_RET_OK on success, or
* \return #RCL_RET_ERROR if any problems occur while hashing.
*/
RCL_PUBLIC
rcl_ret_t
Expand Down
2 changes: 2 additions & 0 deletions rcl/package.xml
Expand Up @@ -19,6 +19,7 @@

<build_export_depend>rmw</build_export_depend>

<depend>libyaml_vendor</depend>
<depend>rcl_interfaces</depend>
<depend>rcl_logging_interface</depend>
<depend>rcl_logging_spdlog</depend> <!-- the default logging impl -->
Expand All @@ -29,6 +30,7 @@
<depend>service_msgs</depend>
<depend>tracetools</depend>
<depend>type_description_interfaces</depend>
<depend>yaml</depend>

<test_depend>ament_cmake_gtest</test_depend>
<test_depend>ament_lint_auto</test_depend>
Expand Down

0 comments on commit 9245a40

Please sign in to comment.