[legacy] catkin plugin: remove default rosdistro
Metadata
Current evaluation
Merged. Removes the EOL Indigo default from the catkin plugin to prevent build failures. Users must now explicitly specify a ROS distribution, resolving LP: #1830784 and ensuring predictable future behavior.
Suggested action: —
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `./runtests.sh static`?
- [x] Have you successfully run `./runtests.sh tests/unit`?
-----
The Catkin plugin currently defaults to using Indigo, a now-EOL ROS distro. Anyone relying on that default is now unable to build a snap. They will probably need to start using another one to get unblocked. Rather than simply set the default to something else and potentially run into this issue again, this PR resolves [LP: #1830784](https://bugs.launchpad.net/snapcraft/+bug/1830784) by getting rid of the default altogether and making users specify which one they want, thus keeping behavior predictable in the future.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged. Removes the EOL Indigo default from the catkin plugin to prevent build failures. Users must now explicitly specify a ROS distribution, resolving LP: #1830784 and ensuring predictable future behavior. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into legacy. Removes the EOL Indigo default from the Catkin plugin, requiring explicit ROS distro specification to prevent future build failures. Resolves Launchpad #1830784. Diff coverage is 100%. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to legacy, the Catkin plugin removes its default EOL Indigo ROS distro. Users must now explicitly specify a distro to avoid build failures. This resolves the issue by enforcing explicit configuration, achieving 100% diff coverage. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1830784 Catkin plugin: stop using EOL indigo as default rosdistro | snapcraft (launchpad) | closed | Closed without resolution, likely abandoned. The request to remove the EOL Indigo default and require explicit user specification was not implemented. | |
| #851 catkin plugin: nicely handle an invalid rosdistro. | snapcraft | merged | Merged changes add rosdistro validation to the catkin plugin, replacing cryptic key errors with clear messages. Approved by reviewers and passed CI. Minor refactoring feedback was noted but did not block the merge. | |
| #850 catkin plugin: add `rosdistro` to help documentation. | snapcraft | merged | Merged documentation update for the catkin plugin adding the rosdistro parameter to snapcraft help catkin output, resolving Launchpad bug 1629955. Approved by one reviewer with three lines added. | |
| #1367 catkin plugin: add support for ROS Lunar | snapcraft | merged | Merged the catkin plugin update to support ROS Lunar alongside Indigo, Jade, and Kinetic. CI initially failed due to a Launchpad bug, resolved by adding lsb-release to build-packages. Approved and merged after passing integration tests. |