Skip to content
This repository has been archived by the owner on Sep 20, 2022. It is now read-only.

updates from buoy_msgs reorg #19

Merged
merged 2 commits into from
Aug 22, 2022
Merged

updates from buoy_msgs reorg #19

merged 2 commits into from
Aug 22, 2022

Conversation

andermi
Copy link
Collaborator

@andermi andermi commented Aug 19, 2022

updates from osrf/mbari_wec_utils#23

buoy_examples is no longer necessary as I've moved the examples to buoy_msgs. Specifically, buoy_api_cpp and buoy_api_py

However, I did leave this in working condition with the reorganized buoy_msgs repo branch

@@ -12,7 +12,8 @@

<depend>rclcpp</depend>
<depend>rclpy</depend>
<depend>buoy_msgs</depend>
<depend>buoy_interfaces</depend>
<depend>buoy_api_cpp</depend>
<depend>std_msgs</depend>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit reorder

@@ -9,11 +9,12 @@ endif()
find_package(ament_cmake REQUIRED)
find_package(rclcpp REQUIRED)
find_package(rclpy REQUIRED)
find_package(buoy_msgs REQUIRED)
find_package(buoy_interfaces REQUIRED)
find_package(buoy_api_cpp REQUIRED)
find_package(std_msgs REQUIRED)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit reorder

@andermi andermi merged commit 4e7253d into main Aug 22, 2022
@andermi andermi deleted the andermi/reorg_buoy_msgs branch August 22, 2022 20:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants