Skip to content

Commit

Permalink
Merge pull request #2097 from jcarpent/topic/ci/ros
Browse files Browse the repository at this point in the history
Add iron and humble in the CI
  • Loading branch information
jcarpent authored Nov 29, 2023
2 parents 6bb26ad + 7b98fae commit 7809577
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 8 deletions.
9 changes: 5 additions & 4 deletions .github/workflows/ros-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,19 +12,20 @@ jobs:
strategy:
matrix:
env:
- {ROS_DISTRO: noetic}
- {ROS_DISTRO: rolling, BUILDER: colcon}
- {ROS_DISTRO: noetic, BUILDER: catkin_tools}
- {ROS_DISTRO: rolling}
- {ROS_DISTRO: iron}
- {ROS_DISTRO: humble}
env:
CCACHE_DIR: /github/home/.ccache # Enable ccache
# The following is a work-around for ROS tooling in conjunction with jrl-cmakemodules:
# The following is a work-around for ROS1 tooling in conjunction with jrl-cmakemodules:
# - catkin_make_isolated has issues with Industrial-CI right now
# - colcon cannot find example-robot-data's python bindings due to ROS1 workspace integration hooks
# - catkin_tools correctly finds example-robot-data in the upstream_ws, but does not execute the 'run_tests' target
# as catkin-tools performs tests as '--catkin-make-args' instead of '--make-args' as colcon and catkin_make_isolated
# The work-around is thus to use catkin_tools for building and sourcing, and to manually specify execution of the test
# target after completion of the regular test target. The output of this step does affect the output of the CI process.
# Note, this does not affect projects that do not have pure CMake projects in their upstream_ws.
BUILDER: catkin_tools
AFTER_RUN_TARGET_TEST: 'ici_with_unset_variables source /root/target_ws/install/setup.bash && cd /root/target_ws/build/pinocchio && make test'
IMMEDIATE_TEST_OUTPUT: 1
runs-on: ubuntu-latest
Expand Down
8 changes: 4 additions & 4 deletions package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,12 @@
<build_depend>doxygen</build_depend>
<doc_depend>doxygen</doc_depend>
<doc_depend>texlive-latex-base</doc_depend>
<!-- Required for the environment variables (ROS_VERSION, ROS_DISTRO) to be available -->
<depend>ros_environment</depend>
<!-- The following tags are recommended by REP-136 -->
<exec_depend condition="$ROS_VERSION == 1">catkin</exec_depend>
<depend condition="$ROS_PYTHON_VERSION == 2">python</depend>
<depend condition="$ROS_PYTHON_VERSION == 3">python3</depend>
<depend condition="$ROS_PYTHON_VERSION == 2">python-numpy</depend>
<depend condition="$ROS_PYTHON_VERSION == 3">python3-numpy</depend>
<depend>python3</depend>
<depend>python3-numpy</depend>
<depend>liburdfdom-dev</depend>
<depend>eigen</depend>
<depend>boost</depend>
Expand Down

0 comments on commit 7809577

Please sign in to comment.