added missing tl_expected dependency (#661)

gaddison-bdai · web-flow · commit d46fdfbdd67c · 2025-05-09T11:44:46.000-04:00
## Change Overview

spot_hardware_interface fails to build due to missing tl_expected header.
Adding `tl_expected` to list of package include dependencies fixes the build error.


## Testing Done

Tested on ubuntu 22.04, outside of docker, with all dependencies installed as specified by readme. Built with 
`colcon build --packages-up-to spot_ros2_control`
diff --git a/spot_hardware_interface/CMakeLists.txt b/spot_hardware_interface/CMakeLists.txt
@@ -22,6 +22,7 @@ set(THIS_PACKAGE_INCLUDE_DEPENDS
   rclcpp
   rclcpp_lifecycle
   spot_msgs
+	tl_expected
 )
 foreach(Dependency IN ITEMS ${THIS_PACKAGE_INCLUDE_DEPENDS})
   find_package(${Dependency} REQUIRED)

Original file line number	Diff line number	Diff line change
`@@ -22,6 +22,7 @@ set(THIS_PACKAGE_INCLUDE_DEPENDS`
`22`	`22`	`rclcpp`
`23`	`23`	`rclcpp_lifecycle`
`24`	`24`	`spot_msgs`
	`25`	`+ tl_expected`
`25`	`26`	`)`
`26`	`27`	`foreach(Dependency IN ITEMS ${THIS_PACKAGE_INCLUDE_DEPENDS})`
`27`	`28`	`find_package(${Dependency} REQUIRED)`