Remove roscore plugin
Metadata
Current evaluation
Removed the duplicate roscore plugin and added a Catkin include-roscore option via merged snapcraft PR #329 (commit d8413cfdb; merge 228b0f4bf).
Resolution: closed resolved
Reason: PR snapcraft#329 was merged, removing the standalone roscore plugin and adding the option to the Catkin plugin. The resolving change is commit d8413cfdb, merged by 228b0f4bf.
Evaluated by: mr-cal-bot
No scores available.
Issue body
The functionality contained within the roscore plugin is completely duplicated in the Catkin plugin. The roscore plugin should be removed, and an option added to the Catkin plugin to determine whether or not to include roscore.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Removed the duplicate roscore plugin and added a Catkin include-roscore option via merged snapcraft PR #329 (commit d8413cfdb; merge 228b0f4bf). | |
| stealth/space-bunny-alpha | — | closed resolved | Removed the duplicate standalone roscore plugin and added Catkin's include-roscore option in merged PR canonical/snapcraft#329. | |
| stealth/space-bunny-alpha | — | closed resolved | PR #329 removed the redundant roscore plugin and added Catkin's include-roscore option; merged as commit 228b0f4bf. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed. The requested changes were implemented: the roscore plugin was removed and its functionality consolidated into the Catkin plugin with a new configuration option. | |
| qwen/qwen3.6-35b-a3b | — | — | Accepted and implemented. The roscore plugin was removed, and its functionality was integrated into the Catkin plugin with a new configuration option. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Request to remove the duplicated roscore plugin and add a Catkin option was closed without implementation. Zero comments indicate the request was likely abandoned or superseded by later project updates. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Likely Fixed By:
snapcraft#329
(confidence 100%)
Merged PR implementing the requested plugin removal and Catkin option.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #329 Remove roscore plugin. | snapcraft | merged | Merged PR #329 (commit d8413cfdb, merge 228b0f4bf) removed the standalone roscore plugin and added an include-roscore option to the Catkin plugin, closing LP bug #1539169. | |
| #1472 catkin plugin: include-roscore is a boolean | snapcraft | merged | Merged PR #1472 (commit 186473104) changed the catkin plugin's include-roscore value from string 'true' to boolean True, correcting its type without functional impact. | |
| #152 Catkin plugin: Add support for ROS tools. | snapcraft | merged | Merged #152 as fc5142126, adding ROS package installation, roslaunch/rosrun support, updated examples, and doubled Catkin test coverage. | |
| #331 Catkin plugin: Handle rosdep empty and multiple dependencies. | snapcraft | merged | PR #331 merged into snapcraft master as commit 1c1119ce3 (merge e3a2b9d03). It refactored the Catkin plugin's rosdep dependency resolution to handle empty output and multiple packages per line, fixing LP bugs #1548406 and #1548404. |