catkin plugin: extract rosdep into new package
Metadata
Current evaluation
Merged in canonical/snapcraft#1392 (99218f59d), extracting rosdep from Catkin into internal.pullers and resolving #1464.
Resolution: closed resolved
Reason: PR #1392 was merged in commit 99218f59d. It moved rosdep into snapcraft.plugins.internal.pullers for reuse by Catkin and the planned Ament/ROS 2 plugin.
Evaluated by: mr-cal-bot
No scores available.
Issue body
This PR resolves #1464 by extracting rosdep out of the catkin plugin and placing it into `snapcraft.plugins.internal.pullers`. This is paving the way for support of Ament, ROS 2's build system, which also uses rosdep.
Fixes #1464
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged in canonical/snapcraft#1392 (99218f59d), extracting rosdep from Catkin into internal.pullers and resolving #1464. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in canonical/snapcraft#1392 (99218f59d), extracting rosdep from Catkin into internal.pullers for reuse by Ament/ROS 2 and resolving #1464. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1392 (commit 99218f59d), extracting rosdep from the Catkin plugin into internal.pullers to enable future Ament/ROS 2 support and closing #1464. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged. Extracts rosdep from the catkin plugin into snapcraft.plugins.internal.pullers to enable future Ament and ROS 2 support. Updates plugin loading to ignore internal packages, resolving issue #1464. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Extracted rosdep from the catkin plugin into a new internal snapcraft.plugins.internal.pullers package to enable ROS 2 support. Updated plugin loading logic to handle internal packages and finalized the architectural changes. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
canonical/snapcraft#1464
(confidence 100%)
The rosdep extraction issue was explicitly fixed and closed by merged PR #1392.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1464 Extract rosdep into its own package | snapcraft | closed | Rosdep was extracted from Catkin into internal.pullers for reuse by Ament/ROS 2; PR canonical/snapcraft#1392 merged in commit 99218f59d. | |
| #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. | |
| #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. | |
| #1548406 Catkin plugin: Can't handle rosdep resolving to nothing | snapcraft (launchpad) | closed | Fixed by canonical/snapcraft#331, merged as commit 1c1119ce3; Catkin now treats empty rosdep output as no dependencies instead of an error. | |
| #1548404 Catkin plugin: Can't handle rosdep resolving to multiple packages | snapcraft (launchpad) | closed | Fixed by ubuntu-core/snapcraft PR #331 (commit 1c1119ce3, 'Catkin plugin: Handle rosdep empty and multiple dependencies'), merged as e3a2b9d03; maintainer ~kyrofa linked the PR when closing the Launchpad bug. | |
| #1581 catkin plugin: support rosdep pip dependencies | snapcraft | merged | Merged PR #1581 in commit 186ae7f91, adding pip-based rosdep dependency support to the Catkin plugin and fixing LP bug #1683036. | |
| #1683036 Catkin plugin doesn't support rosdep dependencies that resolve to pip | snapcraft (launchpad) | closed | Catkin support for rosdep dependencies resolving to pip was merged in snapcraft PR #1581, commit 186ae7f91. | |
| #1714 catkin plugin: recursively merge rosinstall files | snapcraft | closed | Added opt-in recursive rosinstall support to the Catkin plugin, discovering and merging nested rosinstall files in canonical/snapcraft#1934. | |
| #1866718 Snapcraft ROS plugins should be using catkin_pkg instead of fetching all deps with rosdep | snapcraft (launchpad) | closed | Closed as stale without implementation; maintainer ~mr-cal closed it on 2026-07-24 after the ROS 1 plugin proposal remained dormant. | |
| #2833 catkin: remove rospack workaround | snapcraft | merged | Merged PR #2833 in commit 949f90946, removing the Catkin rospack workaround because rosdep can now ignore workspace source packages. |