Skip to content

Commit

Permalink
Fix ament_target_dependencies integration
Browse files Browse the repository at this point in the history
  • Loading branch information
ChrisThrasher committed Dec 21, 2023
1 parent cd1d0ec commit 50a3f3b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 6 deletions.
6 changes: 0 additions & 6 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,9 +21,3 @@ Read the docs [here](https://picknikrobotics.github.io/RSL/files.html).
* [static_vector.hpp](include/rsl/static_vector.hpp) - Static capacity vector class
* [strong_type.hpp](include/rsl/strong_type.hpp) - Strong typedef class
* [try.hpp](include/rsl/try.hpp) - Macro to emulatate absl::CONFIRM or operator? from Rust

## v0 -> v1 API Breaks

No C++ API changes were made between version 0 and version 1, but a change to the build interface was made.
Because we stopped using ament CMake macros, you cannot use `ament_target_dependencies` to link to RSL.
You must instead use `target_link_libraries(... rsl::rsl)` as you would any other non-ament CMake library.
3 changes: 3 additions & 0 deletions cmake/rsl-config.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -7,3 +7,6 @@ find_dependency(tcb_span)
find_dependency(tl_expected)

include(${CMAKE_CURRENT_LIST_DIR}/rsl-targets.cmake)

# Support ament_target_dependencies
set(rsl_TARGETS rsl::rsl)

0 comments on commit 50a3f3b

Please sign in to comment.