catkin plugin: don't stage compilers.
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
Instead of pulling necessary compilers in as stage packages, this PR fixes LP: [#1658828](https://bugs.launchpad.net/snapcraft/+bug/1658828) by pulling them to an alternate working area (similar to rosdep) so they can be used without unnecessarily ending up in the final snap.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Resolved LP #1658828 by routing compilers to an alternate working area instead of staging them, preventing unnecessary inclusion in the final snap. Code coverage increased by 0.02%. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Resolves LP #1658828 by routing compilers to an alternate working area instead of staging them in the catkin plugin, preventing them from being included in the final snap. Code coverage increased by 0.02%. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1658828 Catkin plugin stages compilers | snapcraft (launchpad) | closed | Closed without resolution. The Catkin plugin report highlighted that default compiler staging causes unnecessary snap bloat, but no fix was implemented or merged. | |
| #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. | |
| #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. | |
| #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. | |
| #1779 catkin-tools plugin: use stage-packages | snapcraft | merged | 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. | |
| #840 Catkin plugin: build with in-snap python. | snapcraft | merged | Merged after approval. Resolves LP #1628990 by configuring the catkin plugin to use in-snap Python for both build and run times. Adds gcc, libc-dev, and make as build dependencies. | |
| #881 Catkin plugin: Python nodes require gcc/g++ too. | snapcraft | merged | Merged a fix for Launchpad bug #1638317 updating the Catkin plugin to unconditionally stage gcc and g++ at the beginning. This ensures Python nodes compile correctly regardless of roscpp dependencies. The change modifies two files. |