← Back to issue list

Catkin plugin: Can't handle rosdep resolving to multiple packages

View original Launchpad issue

Metadata

Project
snapcraft (launchpad)
Number
#1548404
Type
issue
State
closed
Author
~kyrofa
Labels
—
Created
Updated
Closed

Current evaluation

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.

Resolution: closed resolved

Reason: The maintainer closed the bug citing PR canonical/snapcraft#331, which landed as commit 1c1119ce3 (merge e3a2b9d03) and made the catkin plugin's rosdep parsing tolerate empty output and multiple packages per line.

Evaluated by: mr-cal-bot

No scores available.

Issue body

The rosdep database contains some entries that will cause rosdep output to contain multiple packages on one line. The Catkin plugin treats that as a single package name and can't successfully download/unpack it.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved 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.
stealth/space-bunny-alpha — closed resolved Catkin rosdep handling was fixed by merged PR canonical/snapcraft#331, adding support for empty output and multiple packages per line.
stealth/space-bunny-alpha — closed resolved Fixed by merged snapcraft PR #331 (merge e3a2b9d03), which updated Catkin to process multiple packages returned on a rosdep output line.
qwen/qwen3.6-35b-a3b — — Closed without resolution. The Catkin plugin fails when rosdep returns multiple packages on a single line, treating it as one invalid name. The issue was abandoned due to zero comments and no proposed fix.
qwen/qwen3.6-35b-a3b — — Closed without resolution. The Catkin plugin incorrectly parses rosdep outputs with multiple packages on one line as a single package name, causing download and unpack failures.

Update history

Date Change
created

Related work

  • Likely Fixed By: canonical/snapcraft#331 (confidence 95%)

    PR referenced by the closing comment; commits 1c1119ce3 / merge e3a2b9d03 implement the rosdep multi-package fix.

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.
89%
#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.
88%
#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.
85%
#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.
83%
#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.
80%
#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.
78%
#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.
77%
#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.
77%
#1629003 Catkin plugin: support kinetic snapcraft (launchpad) closed ROS Kinetic support was added to the Catkin plugin by snapcore/snapcraft PR #842, merged as commit 5d447aedc; the maintainer closed the issue pointing to that fix.
75%
#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).
74%