Remove dependency to docker.io (#985)

* Remove dependency to docker.io The dependency was basically added since this is needed to spawn up ursim. However, the start_ursim script catches the case that no docker executable is available and the tests requiring it are behind a compile definition which is off by default. So, in order to not install docker for users automatically, the dependency shall be removed. (cherry picked from commit 585e29f)
UniversalRobots · May 7, 2024 · beeb8fa · beeb8fa
1 parent c8d961e
commit beeb8fa
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 1 deletion.
diff --git a/.github/workflows/humble-binary-build.yml b/.github/workflows/humble-binary-build.yml
@@ -27,3 +27,4 @@ jobs:
           ROS_DISTRO: ${{ matrix.ROS_DISTRO }}
           ROS_REPO: ${{ matrix.ROS_REPO }}
           CMAKE_ARGS: -DUR_ROBOT_DRIVER_BUILD_INTEGRATION_TESTS=ON
+          ADDITIONAL_DEBS: docker.io netcat-openbsd # Needed for integration tests
diff --git a/.github/workflows/humble-semi-binary-build.yml b/.github/workflows/humble-semi-binary-build.yml
@@ -27,3 +27,4 @@ jobs:
           ROS_DISTRO: ${{ matrix.ROS_DISTRO }}
           ROS_REPO: ${{ matrix.ROS_REPO }}
           CMAKE_ARGS: -DUR_ROBOT_DRIVER_BUILD_INTEGRATION_TESTS=ON
+          ADDITIONAL_DEBS: docker.io netcat-openbsd # Needed for integration tests
diff --git a/ur_robot_driver/package.xml b/ur_robot_driver/package.xml
@@ -59,7 +59,6 @@
   <exec_depend>velocity_controllers</exec_depend>
   <exec_depend>xacro</exec_depend>
 
-  <test_depend>docker.io</test_depend>
   <test_depend>launch_testing_ament_cmake</test_depend>
 
   <export>