catkin-tools plugin: use stage-packages
Metadata
Current evaluation
Merged a fix switching the catkin-tools plugin from build-packages to stage-packages, resolving clean system failures. Added an integration test. Approved by two reviewers and passed CI.
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] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [x] Have you successfully run `./runtests.sh static`?
- [x] Have you successfully run `./runtests.sh unit`?
-----
Currently the catkin-tools plugin uses `build-packages` for its build dependency instead of `stage-packages`, which fails on clean systems.
Use `stage-packages` instead. Also, since this would have been caught by an integration test, add one of those as well (to the snapd suite, specifically).
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged a fix switching the catkin-tools plugin from build-packages to stage-packages, resolving clean system failures. Added an integration test. Approved by two reviewers and passed CI. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged fix for catkin-tools plugin to use stage-packages instead of build-packages, resolving dependency failures on clean systems. Added regression integration test. Approved and merged by maintainer. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix switching the catkin-tools plugin from build-packages to stage-packages, resolving clean system failures. Added an integration test to prevent regression. Approved and merged after passing checks. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2545 catkin plugin: use build-packages for compilers | snapcraft | merged | Merged updates to the Catkin plugin that specify compilers as build-packages instead of isolating them. This resolves a rebuild failure and aligns with current base requirements. Approved by one reviewer and passed CI. | |
| #3168 plugins: fix loading of catkin-tools | snapcraft | merged | Merged a one-line correction changing catkin_tools to catkin-tools in the plugin loader, resolving package loading failures. Approved, passed CI, and addresses LP #1882996. | |
| #1593 catkin tools plugin: add catkin tools support. | snapcraft | merged | Merged a new catkin tools plugin replacing catkin_make. After review, conflict resolution, and adding a beta warning, the changes were approved. Brief testing infrastructure delays preceded successful integration. | |
| #5729 docs: migrate 'the catkin-tools plugin' | snapcraft | merged | Merged documentation update migrating the catkin-tools plugin. Approved by two reviewers with all CI checks passing. Changes span three files, adding 87 lines and removing 5. | |
| #2525 catkin plugin: check stage-snaps for ROS dependencies | snapcraft | merged | Merged to master. Updates the catkin plugin to check stage-snaps for ROS dependencies, preventing redundant fetching by treating them like underlays. Resolves Launchpad bug #1823788. Approved, passed CI, and merged. | |
| #1130 catkin plugin: produce build-ready staging area. | snapcraft | merged | Merged update to the Catkin plugin generating a build-ready staging area by adjusting ROS files post-build, resolving LP #1663385. Approved, passed CI, and modified two files to prevent future parts from referencing the old install directory. | |
| #3282 v2 plugins: add catkin-tools plugin | snapcraft | merged | Merged. Introduces a catkin-tools plugin using the v2 plugin API, reusing shared ROS functionality from existing Catkin and Colcon plugins. Passed static and unit tests, approved by two reviewers, and successfully integrated. | |
| #2459 catkin plugin: describe how to build all packages | snapcraft | merged | Merged a documentation update for the catkin plugin explaining how to build all packages. Adds a missing blurb from a previous bug fix. Approved by two reviewers, passed CI, and modified one file. | |
| #1070 catkin plugin: don't stage compilers. | snapcraft | merged | Merged to master. The catkin plugin now routes compilers to an alternate working area instead of staging them, resolving LP #1658828 and preventing their inclusion in the final snap. Approved, passed CI, and merged. | |
| #234 executor: fix update build stage packages | craft-parts | merged | Merged a fix for build stage package update consistency and resolved unpacked files disappearing after build updates. Approved by two reviewers, passed CI, and integrated with 49 additions and 26 deletions across five files. |