← Back to issue list

plugins: add colcon plugin

View original Github issue

Metadata

Project
snapcraft
Number
#2456
Type
pull request
State
merged
Author
kyrofa
Labels
—
Created
Updated
Closed

Current evaluation

PR #2456 by kyrofa merged (commit cd9516b56) adding a new beta colcon plugin for ROS 2 builds, resolving LP bug #1805213; later extended by #2486, #2593, #2670, #2687 and #2981.

Resolution: closed resolved

Reason: The PR was merged into master as commit cd9516b56 ('colcon plugin: new plugin (#2456)'), adding snapcraft/plugins/colcon.py (2120 lines across 16 files) with 100% diff coverage. It implemented support for the new colcon build system used by ROS 2, closing LP bug #1805213, and was later moved to a plugins package in #2984.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)? - [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)? - [x] Have you successfully run `./runtests.sh static`? - [x] Have you successfully run `./runtests.sh tests/unit`? ----- The newest ROS2 releases introduced a new build system called `colcon`. This PR resolves [LP: #1805213](https://bugs.launchpad.net/snapcraft/+bug/1805213) by adding a new plugin to support it. It leaves it in beta for now until it gets some mileage.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved PR #2456 by kyrofa merged (commit cd9516b56) adding a new beta colcon plugin for ROS 2 builds, resolving LP bug #1805213; later extended by #2486, #2593, #2670, #2687 and #2981.
stealth/space-bunny-alpha — closed resolved Merged PR #2456 added beta colcon build support for ROS 2, resolving LP bug #1805213, with 100% diff coverage.
qwen/qwen3.6-35b-a3b — — Merged the colcon plugin for ROS2 builds, resolving LP: #1805213. Added as a beta feature, the change was approved by two reviewers, passed CI with 100% diff coverage, and introduced 2120 lines across 16 files.
qwen3.6-35b-a3b-mtp-q6 — — Merged a new colcon plugin for ROS2 builds, resolving LP: #1805213. Released as beta, it passed all unit tests and increased overall code coverage.

Update history

No update history recorded yet.

Related work

  • Related To: snapcraft#2486 (confidence 92%)

    Follow-up commit 60c3c9b18 'colcon plugin: support build-time chaining' extending the new plugin.

  • Related To: snapcraft#2687 (confidence 90%)

    Commit bdc2150bb 'colcon plugin: add ability to ignore packages'.

  • Related To: snapcraft#2981 (confidence 85%)

    Commit e54d90b1b 'colcon plugin: rewrite poco absolute library paths'.

  • Related To: snapcraft#2984 (confidence 80%)

    Commit 38996d538 relocated the plugin into the new plugins package.

Related issues

Issue Project State Summary Similarity
#1805213 Support the new ros2 build tool colcon snapcraft (launchpad) closed Added and merged the beta colcon plugin for ROS 2 builds in snapcore/snapcraft#2456 (commit cd9516b56), resolving LP bug #1805213.
81%
#3268 v2 plugins: add catkin plugin snapcraft merged PR #3268 by kyrofa merged as commit d0f4177e9, adding the v2 Catkin plugin and refactoring shared ROS functionality out of the Colcon plugin; spread tests followed in #3282.
78%
#3097 colcon v2 plugin + ros2 extension snapcraft closed Closed unmerged and superseded by #3203, which added experimental ROS 2 Foxy support and the colcon v2 plugin in commit 88cc08fad.
77%
#4054 core22 colcon plugin: re-source environment after installation (#3994) snapcraft merged Backported the core22 colcon fix in #3994: source ROS 2 after dependency installation. PR #4054 merged via d634246 into hotfix/7.3, with new spread coverage.
77%
#3113 colcon: use packages.ros.org/ros2 repository snapcraft merged Merged PR #3113 (commit d76f466f6) switched the colcon plugin from the broken repo.ros2.org apt source to packages.ros.org/ros2.
76%
#3203 experimental ros2 extension & colcon v2 plugin snapcraft merged Merged experimental ROS 2 Foxy support and the colcon v2 plugin in #3203 (commit 88cc08fad), replacing the earlier unmerged #3097 attempt.
75%
#3994 core22 colcon plugin: re-source environment after installation snapcraft merged Merged #3994, moving ROS 2 environment sourcing until after colcon dependencies are installed. It also added package-state cleanup and regression spread coverage for custom ROS 2 interfaces.
74%
#2593 colcon plugin: add support for dashing snapcraft merged Merged ROS 2 Dashing support into the colcon plugin via #2593, committed as e00b6903e.
74%
#3750 Add core22 colcon plugin / ros2-humble extension snapcraft merged Merged into main as commit 2013e0e18, adding the core22 colcon plugin and ros2-humble extension.
74%
#2835 colcon plugin: support ROS 2 Eloquent snapcraft merged Merged ROS 2 Eloquent Elusor support for the colcon plugin in PR #2835 via commit 2252dd78e.
73%