New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add dependency to force_torque_sensor_broadcaster #208

Merged

bmagyar merged 1 commit into ros-controls:master from nkalupahana:patch-1

Jun 28, 2021

Contributor

nkalupahana commented Jun 27, 2021

force_torque_sensor_broadcaster uses realtime_tools, but it isn't marked as a dependency in its package.xml. This can cause builds to fail if colcon decides to build realtime_tools after force_torque_sensor_broadcaster (see CI: https://github.com/nkalupahana/ros2-foxy-macos/runs/2924666462?check_suite_focus=true)

This PR adds realtime_tools as a dependency, thus fixing this build issue. (CI: https://github.com/nkalupahana/ros2-foxy-macos/runs/2926064882?check_suite_focus=true)


          Update package.xml

6ea3d6a

Member

bmagyar commented Jun 28, 2021

Great find, thanks!

bmagyar approved these changes

View reviewed changes

bmagyar merged commit e72f39f into ros-controls:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment