Catkin plugin: Confusing print when dependency is an invalid package
Metadata
Current evaluation
Catkin now reports a clear error for invalid dependency packages; fixed by merged canonical/snapcraft PR #330 (e44ded923).
Resolution: closed resolved
Reason: PR #330 added explicit handling and a clear error message for invalid Catkin dependencies. It was merged as d04c07320, with implementation commit e44ded923.
Evaluated by: mr-cal-bot
No scores available.
Issue body
The Catkin plugin interrogates the source code being built for dependencies and fetches them. If those dependencies happen to resolve to a non-existing package, the error message simply prints the name of the package-- it doesn't actually say what the problem was.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Catkin now reports a clear error for invalid dependency packages; fixed by merged canonical/snapcraft PR #330 (e44ded923). | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #330 (commit e44ded923) added explicit handling and a clear error when a Catkin dependency is not a valid package. | |
| stealth/space-bunny-alpha | — | closed resolved | Catkin invalid-dependency errors were clarified in merged PR #330 (commit e44ded923), which added an explicit invalid-package error message. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without resolution. The confusing error message for invalid Catkin dependencies remains unaddressed. | |
| qwen/qwen3.6-35b-a3b | — | — | Abandoned and closed without resolution. The report detailed confusing error output in the Catkin plugin when dependencies resolve to invalid packages, but received no comments or follow-up activity. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Likely Fixed By:
canonical/snapcraft#330
(confidence 100%)
Merged PR directly fixing Launchpad bug #1548370 in commit e44ded923.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #330 Catkin plugin: Print nice message if dependency is invalid. | snapcraft | merged | Merged PR #330 via d04c07320, with implementation commit e44ded923, catches invalid Catkin dependency errors and prints a clear explanation, resolving Launchpad bug #1548370. | |
| #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. | |
| #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. | |
| #1791743 Catkin plugin doesn't follow project error guidelines | snapcraft (launchpad) | closed | Catkin error handling was aligned with Snapcraft guidelines in merged PR #2255, commit c8017788d, replacing built-in exceptions with SnapcraftException. | |
| #175 Catkin plugin: Refactor build. | snapcraft | merged | Merged in snapcraft#175 via commit 9045874fd, refactoring Catkin environment, dependency handling, build logic, source-space support, and tests. | |
| #1552498 catkin plugin uses unhashables for dependencies | snapcraft (launchpad) | closed | Fixed the Catkin unhashable-list dependency error by merging ubuntu-core/snapcraft#363 in commit 8add8448b. |