Catkin plugin: should be using release by default
Metadata
Current evaluation
Catkin's default CMake build type was fixed by snapcore/snapcraft PR #1470, confirmed by the maintainer.
Resolution: closed resolved
Reason: Maintainer kyrofa linked PR #1470 as the fix, matching the request to default Catkin builds to Release.
Evaluated by: mr-cal-bot
No scores available.
Issue body
The catkin plugin doesn't set the cmake build type. It should set it, and default to Release. It should also support a "debug" option, which is a boolean defaulting to false. If true, set the build type to Debug.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Catkin's default CMake build type was fixed by snapcore/snapcraft PR #1470, confirmed by the maintainer. | |
| stealth/space-bunny-alpha | — | closed resolved | Catkin now defaults to CMake Release builds and supports Debug via build attributes; merged in snapcore/snapcraft#1470 via commit e41b00bbb. | |
| stealth/space-bunny-alpha | — | closed resolved | Catkin now defaults to Release builds, with Debug selectable through build-attributes. Fix merged in canonical/snapcraft#1470 as e41b00bbb. | |
| qwen/qwen3.6-35b-a3b | — | — | Requesting the catkin plugin default to a Release CMake build type with an optional debug flag was closed without implementation or discussion. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without implementation. The catkin plugin was never updated to default to Release or add a debug build type option. No fix, merge, or resolution occurred. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without implementation or comments, likely abandoned. The requested default Release build type and debug option for the catkin plugin were not added. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Request to set catkin plugin CMake build type to Release by default and add a debug option was closed without implementation. No fixes or pull requests were submitted, indicating the request was abandoned due to inactivity. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Likely Fixed By:
snapcore/snapcraft#1470
(confidence 99%)
The maintainer's sole closing comment explicitly identifies this PR as the fix.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1470 catkin plugin: default to release build | snapcraft | merged | Merged #1470 in commit e41b00bbb, making the catkin plugin default to Release builds and allowing Debug builds through build-attributes. | |
| #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. | |
| #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. | |
| #1827148 Catkin rebuilding always starts from scratch | snapcraft (launchpad) | closed | Catkin incremental rebuilds were fixed by stabilizing CMake arguments so unchanged components can be reused; PR snapcore/snapcraft#2545 landed as commit ddb2d2627. | |
| #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. |