← Back to issue list

Catkin plugin: Print nice message if dependency is invalid.

View original Github issue

Metadata

Project
snapcraft
Number
#330
Type
pull request
State
merged
Author
kyrofa
Labels
Created
Updated
Closed

Current evaluation

Merged. Catches repo.Ubuntu exceptions for invalid catkin dependencies and prints a clear error message, resolving Launchpad bug #1548370. Approved by @sergiusens following a successful CI retrigger.

Suggested action:

No scores available.

Issue body

This PR fixes LP: [#1548370](https://bugs.launchpad.net/snapcraft/+bug/1548370) by catching the relevant exception raised by `repo.Ubuntu` and printing a message explaining exactly what the problem was.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged. Catches repo.Ubuntu exceptions for invalid catkin dependencies and prints a clear error message, resolving Launchpad bug #1548370. Approved by @sergiusens following a successful CI retrigger.
qwen3.6-35b-a3b-mtp-q6 Merged. Catches repo.Ubuntu exceptions and prints a descriptive error for invalid Catkin dependencies, resolving Launchpad bug #1548370. Approved by sergiusens following a successful build retrigger.
qwen3.6-35b-a3b-mtp-q6 Merged after successful CI retrigger. Catches repo.Ubuntu exceptions for invalid catkin dependencies and prints a clear error message, resolving Launchpad bug #1548370.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1548370 Catkin plugin: Confusing print when dependency is an invalid package snapcraft (launchpad) closed Closed without resolution. The confusing error message for invalid Catkin dependencies remains unaddressed.
75%
#331 Catkin plugin: Handle rosdep empty and multiple dependencies. snapcraft merged Merged refactoring of the Catkin plugin rosdep resolver to handle empty results and multiple dependencies per line. Fixes Launchpad bugs #1548406 and #1548404. Approved by maintainer sergiusens.
71%