Option for Catkin Build for ROS 1 Projects

ros-tooling / action-ros-ci

Github Action to build and test ROS 2 packages using colcon

Apache License 2.0

145 stars 53 forks source link

Option for Catkin Build for ROS 1 Projects #800

Open Shobuj-Paul opened 1 year ago

Shobuj-Paul commented 1 year ago

Description

Catkin and Colcon build systems differ in their CMake implementations, and hence not completely back compatible. So a catkin_tools implementation is required for ROS-1 projects.

Completion Criteria

[ ] Have the container install and run catkin build system instead of colcon build system.
[ ] Testing on a few basic projects which are known to build locally with catkin.

Implementation Notes / Suggestions

A choice between setting the build system would be helpful for CI implementation depending on the project.

Testing Notes / Suggestions

Widely known ROS projects which build locally with catkin can be tested such as TurtleBot3, Moveit, and MAVROS.

christophebedard commented 1 year ago

We're happy to review a PR that adds this functionality!

You could add a use-catkin option. If true, some colcon-specific options such as colcon-defaults, colcon-mixin-repository, and colcon-extra-args would not have any effect.