Skip to content

Commit

Permalink
Depend on launch packages instead of ros_testing to avoid circular de…
Browse files Browse the repository at this point in the history
…pendency (#685)

* Depend on launch packages instead of ros_testing to avoid circular dependency

Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>

* packages don't depend on launch_ros

Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>
  • Loading branch information
sloretz authored Dec 24, 2021
1 parent 05f2ae7 commit 298764c
Show file tree
Hide file tree
Showing 8 changed files with 29 additions and 8 deletions.
4 changes: 3 additions & 1 deletion ros2action/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,10 @@
<test_depend>ament_flake8</test_depend>
<test_depend>ament_pep257</test_depend>
<test_depend>ament_xmllint</test_depend>
<test_depend>launch</test_depend>
<test_depend>launch_testing</test_depend>
<test_depend>launch_testing_ros</test_depend>
<test_depend>python3-pytest</test_depend>
<test_depend>ros_testing</test_depend>
<test_depend>test_msgs</test_depend>

<export>
Expand Down
5 changes: 4 additions & 1 deletion ros2doctor/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,11 @@
<test_depend>ament_flake8</test_depend>
<test_depend>ament_pep257</test_depend>
<test_depend>ament_xmllint</test_depend>
<test_depend>launch</test_depend>
<test_depend>launch_ros</test_depend>
<test_depend>launch_testing</test_depend>
<test_depend>launch_testing_ros</test_depend>
<test_depend>python3-pytest</test_depend>
<test_depend>ros_testing</test_depend>
<test_depend>std_msgs</test_depend>

<export>
Expand Down
4 changes: 3 additions & 1 deletion ros2interface/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,10 @@
<test_depend>ament_flake8</test_depend>
<test_depend>ament_pep257</test_depend>
<test_depend>ament_xmllint</test_depend>
<test_depend>launch</test_depend>
<test_depend>launch_testing</test_depend>
<test_depend>launch_testing_ros</test_depend>
<test_depend>python3-pytest</test_depend>
<test_depend>ros_testing</test_depend>
<test_depend>ros2cli_test_interfaces</test_depend>
<test_depend>test_msgs</test_depend>

Expand Down
5 changes: 4 additions & 1 deletion ros2lifecycle/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,11 @@
<test_depend>ament_flake8</test_depend>
<test_depend>ament_pep257</test_depend>
<test_depend>ament_xmllint</test_depend>
<test_depend>launch</test_depend>
<test_depend>launch_ros</test_depend>
<test_depend>launch_testing</test_depend>
<test_depend>launch_testing_ros</test_depend>
<test_depend>python3-pytest</test_depend>
<test_depend>ros_testing</test_depend>
<test_depend>ros2lifecycle_test_fixtures</test_depend>

<export>
Expand Down
5 changes: 4 additions & 1 deletion ros2node/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -22,9 +22,12 @@
<test_depend>ament_flake8</test_depend>
<test_depend>ament_pep257</test_depend>
<test_depend>ament_xmllint</test_depend>
<test_depend>launch</test_depend>
<test_depend>launch_ros</test_depend>
<test_depend>launch_testing</test_depend>
<test_depend>launch_testing_ros</test_depend>
<test_depend>python3-pytest</test_depend>
<test_depend>rclpy</test_depend>
<test_depend>ros_testing</test_depend>
<test_depend>test_msgs</test_depend>

<export>
Expand Down
4 changes: 3 additions & 1 deletion ros2pkg/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,10 @@
<test_depend>ament_flake8</test_depend>
<test_depend>ament_pep257</test_depend>
<test_depend>ament_xmllint</test_depend>
<test_depend>launch</test_depend>
<test_depend>launch_testing</test_depend>
<test_depend>launch_testing_ros</test_depend>
<test_depend>python3-pytest</test_depend>
<test_depend>ros_testing</test_depend>

<export>
<build_type>ament_python</build_type>
Expand Down
5 changes: 4 additions & 1 deletion ros2service/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,11 @@
<test_depend>ament_flake8</test_depend>
<test_depend>ament_pep257</test_depend>
<test_depend>ament_xmllint</test_depend>
<test_depend>launch</test_depend>
<test_depend>launch_ros</test_depend>
<test_depend>launch_testing</test_depend>
<test_depend>launch_testing_ros</test_depend>
<test_depend>python3-pytest</test_depend>
<test_depend>ros_testing</test_depend>
<test_depend>test_msgs</test_depend>

<export>
Expand Down
5 changes: 4 additions & 1 deletion ros2topic/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,11 @@
<test_depend>ament_pep257</test_depend>
<test_depend>ament_xmllint</test_depend>
<test_depend>geometry_msgs</test_depend>
<test_depend>launch</test_depend>
<test_depend>launch_ros</test_depend>
<test_depend>launch_testing</test_depend>
<test_depend>launch_testing_ros</test_depend>
<test_depend>python3-pytest</test_depend>
<test_depend>ros_testing</test_depend>
<test_depend>std_msgs</test_depend>
<test_depend>test_msgs</test_depend>

Expand Down

0 comments on commit 298764c

Please sign in to comment.