Skip to content

Commit

Permalink
Exclude other RMW from Github CI build
Browse files Browse the repository at this point in the history
Since ros2#145, the CI build of rmw_cyclonedds_cpp has been failing on Windows due to inadvertently injecting fastrtps into the build process.
fastrtps fails to build (eProsima/Fast-DDS#1173) causing the CI to fail.
There doesn't seem to be a better way to suppress this in action-ros-ci ros-tooling/action-ros-ci#177

Fixes ros2#164
  • Loading branch information
rotu committed Apr 28, 2020
1 parent 3c306d3 commit 3dc89b8
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 3 deletions.
11 changes: 11 additions & 0 deletions .github/resources/suppress_other_rmw.repos
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
definitions:
- &empty_repo
type: zip
url: data:application/zip;base64,UEsFBgAAAAAAAAAAAAAAAAAAAAAAAA==

repositories:
ros2/rosidl_typesupport_connext/COLCON_IGNORE: *empty_repo
ros2/rmw_connext/COLCON_IGNORE: *empty_repo

ros2/rosidl_typesupport_fastrtps/COLCON_IGNORE: *empty_repo
ros2/rmw_fastrtps/COLCON_IGNORE: *empty_repo
8 changes: 5 additions & 3 deletions .github/workflows/CI.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,11 @@ jobs:
# azure ubuntu repo can be flaky so add an alternate source
run: sed -e 's/azure.archive.ubuntu.com/us.archive.ubuntu.com/g' -e t -e d /etc/apt/sources.list | sudo tee /etc/apt/sources.list.d/nonazure.list
- name: Acquire ROS dependencies
uses: ros-tooling/[email protected].19
uses: ros-tooling/[email protected].20
- name: Build and test ROS
uses: ros-tooling/[email protected].15
uses: ros-tooling/[email protected].16
with:
package-name: rmw_cyclonedds_cpp
vcs-repo-file-url: https://raw.githubusercontent.com/ros2/ros2/${{ matrix.rosdistro }}/ros2.repos
vcs-repo-file-url: >
https://raw.githubusercontent.com/ros2/ros2/${{ matrix.rosdistro }}/ros2.repos
https://raw.githubusercontent.com/${{github.repository}}/${{github.sha}}/.github/resources/suppress_other_rmw.repos

0 comments on commit 3dc89b8

Please sign in to comment.