catkin plugin: recursively merge rosinstall files
Metadata
Current evaluation
Added opt-in recursive rosinstall support to the Catkin plugin, discovering and merging nested rosinstall files in canonical/snapcraft#1934.
Resolution: closed resolved
Reason: PR canonical/snapcraft#1934 was merged as commit 6bb63bb25 and fixes #1714. It added an opt-in recursive-rosinstall option for discovering nested workspace repositories.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Estimate: 4 days
The Catkin plugin currently supports parsing rosinstall files to pull the various repositories that make up the workspace that it will build. A request from Shadow Robotics is to add the ability to then look inside the pulled workspace for more rosinstall files, and fetch their repositories as well. This is not supported by rosinstall alone. This will require us to have a concept of being finished. How do we know that we’ve finished pulling everything necessary if we’re doing so in a recursive manner?
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Added opt-in recursive rosinstall support to the Catkin plugin, discovering and merging nested rosinstall files in canonical/snapcraft#1934. | |
| stealth/space-bunny-alpha | — | closed resolved | Added opt-in recursive rosinstall support to the Catkin plugin via merged PR canonical/snapcraft#1934, enabling discovery of nested rosinstall files. | |
| qwen/qwen3.6-35b-a3b | — | — | The request to recursively merge rosinstall files in the catkin plugin was closed without implementation or discussion. The feature was not added and the issue appears abandoned. | |
| qwen/qwen3.6-35b-a3b | — | — | The catkin plugin feature to recursively merge rosinstall files was closed and abandoned without implementation. Recursive fetching logic was never added due to unresolved design questions and lack of discussion. | |
| qwen/qwen3.6-35b-a3b | — | — | The request to recursively merge rosinstall files in the catkin plugin was closed without implementation. With zero comments and no linked resolution, the feature was abandoned after over a year of inactivity. | |
| qwen/qwen3.6-35b-a3b | — | — | A request to add recursive rosinstall file merging to the catkin plugin was closed without implementation. With zero comments and over eight years of inactivity, the feature was abandoned. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | The request to recursively merge rosinstall files in the catkin plugin was closed without implementation. Zero comments and no associated pull requests indicate the feature was abandoned. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
canonical/snapcraft#1934
(confidence 100%)
Merged implementation of recursive rosinstall support in commit 6bb63bb25.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1934 catkin plugin: support recursive rosinstall files | snapcraft | merged | Merged PR #1934 (commit 6bb63bb25) adds an opt-in `recursive-rosinstall` option to the Catkin plugin, merging nested rosinstall files from fetched repos. Fixes issue #1714; tested on the stable/pr-1934-recursive-rosinstall channel. | |
| #1314 catkin plugin: add support for rosinstall files | snapcraft | merged | Merged PR #1314 as commit 97a416d0a, adding ordered multi-file rosinstall support to the Catkin plugin and resolving Launchpad #1533735. | |
| #1533735 Catkin plugin: Support rosinstall files | snapcraft (launchpad) | closed | Catkin rosinstall support was merged in PR #1314 (commit 97a416d0a); rosinstall-files was later corrected to a pull property in #1473. | |
| #1473 catkin plugin: rosinstall-files is a pull property | snapcraft | merged | Merged PR #1473 corrected the catkin plugin to treat rosinstall-files as a pull property, committed as 09fcddf13. | |
| #1715 catkin plugin: detect and gracefully handle rosinstall clashes | snapcraft | closed | Abandoned as unnecessary: recursive rosinstall fetching was supported via #1934 (6bb63bb25), and @kyrofa said clash detection offered no gain. | |
| #1392 catkin plugin: extract rosdep into new package | snapcraft | merged | Merged in canonical/snapcraft#1392 (99218f59d), extracting rosdep from Catkin into internal.pullers and resolving #1464. | |
| #152 Catkin plugin: Add support for ROS tools. | snapcraft | merged | Merged #152 as fc5142126, adding ROS package installation, roslaunch/rosrun support, updated examples, and doubled Catkin test coverage. | |
| #331 Catkin plugin: Handle rosdep empty and multiple dependencies. | snapcraft | merged | PR #331 merged into snapcraft master as commit 1c1119ce3 (merge e3a2b9d03). It refactored the Catkin plugin's rosdep dependency resolution to handle empty output and multiple packages per line, fixing LP bugs #1548406 and #1548404. | |
| #163 Catkin plugin: use rosdep for dependency resolution | snapcraft | merged | Merged in PR #163 (commit 87b2e24f6), replacing manual Catkin package.xml dependency parsing with rosdep; PR #183 backported the change to Snapcraft 1.x. | |
| #2525 catkin plugin: check stage-snaps for ROS dependencies | snapcraft | merged | Merged #2525 as d1c5f1b00, updating the catkin plugin to honor ROS dependencies supplied by stage-snaps and avoid redundant fetches. |