Skip to content

Commit da01094

Browse files
mergify[bot]Felix Exner (fexner)
andauthored
Remove dependency to docker.io (backport of #985) (#987)
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) Co-authored-by: Felix Exner (fexner) <[email protected]>
1 parent c8d961e commit da01094

File tree

3 files changed

+2
-1
lines changed

3 files changed

+2
-1
lines changed

.github/workflows/humble-binary-build.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,3 +27,4 @@ jobs:
2727
ROS_DISTRO: ${{ matrix.ROS_DISTRO }}
2828
ROS_REPO: ${{ matrix.ROS_REPO }}
2929
CMAKE_ARGS: -DUR_ROBOT_DRIVER_BUILD_INTEGRATION_TESTS=ON
30+
ADDITIONAL_DEBS: docker.io netcat-openbsd # Needed for integration tests

.github/workflows/humble-semi-binary-build.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,3 +27,4 @@ jobs:
2727
ROS_DISTRO: ${{ matrix.ROS_DISTRO }}
2828
ROS_REPO: ${{ matrix.ROS_REPO }}
2929
CMAKE_ARGS: -DUR_ROBOT_DRIVER_BUILD_INTEGRATION_TESTS=ON
30+
ADDITIONAL_DEBS: docker.io netcat-openbsd # Needed for integration tests

ur_robot_driver/package.xml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,6 @@
5959
<exec_depend>velocity_controllers</exec_depend>
6060
<exec_depend>xacro</exec_depend>
6161

62-
<test_depend>docker.io</test_depend>
6362
<test_depend>launch_testing_ament_cmake</test_depend>
6463

6564
<export>

0 commit comments

Comments
 (0)