Catkin plugin: Can't handle rosdep resolving to nothing
Metadata
Current evaluation
Fixed by canonical/snapcraft#331, merged as commit 1c1119ce3; Catkin now treats empty rosdep output as no dependencies instead of an error.
Resolution: closed resolved
Reason: The maintainer linked canonical/snapcraft#331, which was merged as commit 1c1119ce3 and updated rosdep handling to accept empty output.
Evaluated by: mr-cal-bot
No scores available.
Issue body
The rosdep database contains some entries that specifically resolve to nothing. The Catkin plugin treats this as an error, when it really means there's just nothing to install.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Fixed by canonical/snapcraft#331, merged as commit 1c1119ce3; Catkin now treats empty rosdep output as no dependencies instead of an error. | |
| stealth/space-bunny-alpha | — | closed resolved | Catkin plugin treated empty rosdep resolution as an error; fixed by merged PR canonical/snapcraft#331 (commit 1c1119ce3, merge e3a2b9d03) refactoring rosdep resolution to handle empty output. | |
| stealth/space-bunny-alpha | — | closed resolved | Catkin now treats an empty rosdep resolution as nothing to install rather than an error. Fixed by merged canonical/snapcraft#331 (e3a2b9d03). | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without comments or resolution. The Catkin plugin issue regarding rosdep entries resolving to nothing was unaddressed, indicating the report was abandoned or resolved internally without a code change. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without resolution or discussion. The Catkin plugin incorrectly treats rosdep entries resolving to nothing as errors instead of skipping them. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Likely Fixed By:
canonical/snapcraft#331
(confidence 100%)
Merged fix for empty and multi-package rosdep resolutions.
-
Related To:
snapcraft (launchpad)#1548404
(confidence 100%)
Companion Catkin rosdep bug fixed by the same PR.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #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. | |
| #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. | |
| #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. | |
| #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. | |
| #1823788 Catkin plugin doesn't notice ROS dependencies satisfied by stage-snaps | snapcraft (launchpad) | closed | Catkin stage-snap dependency detection was implemented by merged snapcraft PR #2525 (commit d1c5f1b00), avoiding redundant ROS dependency fetches. | |
| #1832044 Catkin plugin doesn't check the workspace for dependencies | snapcraft (launchpad) | closed | Catkin plugin failed on whole-workspace builds because rosdep (#685) couldn't resolve deps satisfied by in-workspace packages. Workaround merged in snapcraft PR #2585 (commit b88e37814); the workaround was later dropped in #2833 once rosdep was fixed upstream. | |
| #1548370 Catkin plugin: Confusing print when dependency is an invalid package | snapcraft (launchpad) | closed | Catkin now reports a clear error for invalid dependency packages; fixed by merged canonical/snapcraft PR #330 (e44ded923). | |
| #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. | |
| #183 Backport to 1.x: Catkin plugin: use rosdep for dependency resolution. | snapcraft | merged | Merged into 1.x via PR #183, backporting PR #163 so Catkin uses rosdep instead of manually parsing package.xml dependencies. |