Transition catkin plugin to use catkin build
Metadata
Current evaluation
Superseded by merged PR canonical/snapcraft#1593 (7e31fa0dd), which added a separate beta catkin-tools plugin using catkin build while preserving catkin_make behavior.
Resolution: closed superseded
Reason: PR #1593 introduced catkin build support as a new catkin-tools plugin rather than changing the existing plugin. The maintainer confirmed the approach avoided compatibility issues and was available for testing in edge ahead of Snapcraft 2.36.
Evaluated by: mr-cal-bot
No scores available.
Issue body
The catkin plugin is currently using the catkin_make_isolated method to create the desired files needed for the snap. However, the current recommended method for constructing ros packages is using catkin build. Using catkin build would provide a more robust method for constructing the packages.
More information about the differences between make and build can be found here: https://catkin-tools.readthedocs.io/en/latest/migration.html
In the plugin source there are also some mentions of transitioning to catkin Tools.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed superseded | Superseded by merged PR canonical/snapcraft#1593 (7e31fa0dd), which added a separate beta catkin-tools plugin using catkin build while preserving catkin_make behavior. | |
| stealth/space-bunny-alpha | — | closed superseded | Catkin Tools support was added as a separate beta plugin in merged PR #1593 (7e31fa0dd), replacing the proposed transition of the existing catkin plugin. | |
| stealth/space-bunny-alpha | — | closed stale | Closed without a recorded Catkin-plugin migration; catkin build support was added as a separate beta plugin in PR #1593. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without implementation; likely abandoned or superseded. The proposal to replace catkin_make_isolated with catkin build in the plugin was not addressed or merged. | |
| qwen/qwen3.6-35b-a3b | — | — | Requesting a transition from catkin_make_isolated to catkin build was closed without implementation. Zero comments and extended inactivity indicate the suggestion was abandoned. | |
| qwen/qwen3.6-35b-a3b | — | — | The suggestion to transition the catkin plugin from catkin_make_isolated to catkin build was closed without implementation or discussion. The request remains unaddressed and effectively abandoned. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Abandoned. The proposal to transition the catkin plugin from catkin_make_isolated to catkin build was closed without discussion or implementation. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Superseded By:
canonical/snapcraft#1593
(confidence 100%)
Merged PR #1593 added the separate catkin-tools implementation requested by the issue.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1593 catkin tools plugin: add catkin tools support. | snapcraft | merged | Merged PR #1593 (commit 7e31fa0dd), adding a beta catkin-tools plugin that uses catkin tools for builds while retaining catkin_make behavior. | |
| #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. | |
| #1650628 Update catkin plugin to use get_pull_properties | snapcraft (launchpad) | closed | Merged snapcraft#1001 in commit 561a04341, updating Catkin to use get_pull_properties() instead of modifying the schema and adding a unit test. | |
| #840 Catkin plugin: build with in-snap python. | snapcraft | merged | Merged #840 as 7f9fa3ac1, fixing LP #1628990 by using in-snap Python during Catkin builds and adding gcc, libc-dev, and make as build packages. | |
| #1794778 Update the catkin plugin to be base aware | snapcraft (launchpad) | closed | Base-aware catkin and catkin-tools support was merged in snapcraft#2333, commit a7fb1b137. | |
| #1721168 Catkin plugin: support building the entire workspace | snapcraft (launchpad) | closed | Implemented in snapcraft PR #1743 (commit ec4cac26a, "catkin plugin: support building entire workspace"), merged ~Nov 2017. catkin-packages became optional: omitting it builds the whole workspace, while an empty list still means no packages. | |
| #2255 catkin plugin: use SnapcraftException | snapcraft | merged | Merged PR #2255 as c8017788d, replacing catkin plugin built-in exceptions with SnapcraftException and fixing LP #1791743. | |
| #1628990 Catkin plugin: example doesn't build on vanilla install | snapcraft (launchpad) | closed | Fixed by merged PR snapcore/snapcraft#840 (commit 7f9fa3ac1): Catkin builds now use in-snap Python and add gcc, libc-dev and make as build-packages, so the example builds on a vanilla system. Closed by maintainer ~kyrofa. |