← Back to issue list

craftctl: add snapcraftctl compatibility wrapper

View original Github issue

Metadata

Project
snapcraft
Number
#3751
Type
pull request
State
merged
Author
cmatsuoka
Labels
—
Created
Updated
Closed

Current evaluation

Merged #3751 added a snapcraftctl compatibility wrapper delegating to craftctl, prompted core22 users to migrate, and restored execution of existing spread tests.

Resolution: closed resolved

Reason: Merged via commit bd286224d, with implementation commit e973755fb, adding the wrapper and migration guidance while fixing core22 spread-test execution.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Add snapcraftctl wrapper to craftctl to make the transition to core22 easier. Execution of the wrapper displays a message encouraging the user to update to craftctl. Also add a spread test and fix existing core22 spread tests that were not being executed. Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com> - [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/)? - [ ] Have you successfully run `make lint`? - [ ] Have you successfully run `pytest tests/unit`? ----- CRAFT-965

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged #3751 added a snapcraftctl compatibility wrapper delegating to craftctl, prompted core22 users to migrate, and restored execution of existing spread tests.
stealth/space-bunny-alpha — closed resolved Merged #3751 adds a craftctl snapcraftctl compatibility wrapper for the core22 transition, including a migration message and spread-test fixes.
stealth/space-bunny-alpha — closed resolved Merged #3751 (commit e973755) to add a snapcraftctl compatibility wrapper to craftctl, easing the core22 transition; it prompts users to update and adds spread coverage.
qwen/qwen3.6-35b-a3b — — Merged a snapcraftctl compatibility wrapper into craftctl to ease the core22 transition. The wrapper outputs an update message and includes spread tests. Approved by reviewer, passed CI, and merged into main.
qwen3.6-35b-a3b-mtp-q6 — — Merged. Adds a snapcraftctl wrapper to craftctl for core22 transition, including spread test updates and fixes for unexecuted core22 tests.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#5774 Remove all snapcraftctl code and compatibility wrapper for core22 snapcraft closed PR to strip snapcraftctl code for core22 was closed unmerged: maintainers (mr-cal) said removal can't happen for core22/core24 as it would break existing builds, so the wrapper was kept. Later #5983 (eaadefa01) errors on snapcraftctl only in core26.
77%
#5754 Remove the `snapcraftctl` compatibility wrapper snapcraft closed Closed as won't-fix by mr-cal: snapcraftctl wrapper must stay for core22/core24 to avoid breaking existing builds. Implementation PR #5774 was closed unmerged; #5983 (eaadefa01) only makes core26 error on snapcraftctl.
76%
#1091 docs: revamp craftctl how-to craft-parts merged Merged PR #1091 as 44aa6508, revamping the craftctl how-to with upstream Snapcraft guidance and adding a 2.10.0 changelog entry.
75%
#5983 feat!: error on snapcraftctl snapcraft merged Merged #5983 as eaadefa01: snapcraftctl now errors in core26+ override scripts, and related references were renamed to craftctl.
74%
#2002 many: add snapcraftctl command for scriptlets snapcraft merged Merged PR #2002 (commit bbca282fa) added the snapcraftctl CLI for scriptlets to talk to the parent Snapcraft process, including `snapcraftctl build` and the override-build scriptlet. Resolved #1675 and LP #1751749.
74%
#6336 Snapcraft 9: snapcraftctl errors in core22 snaps snapcraft closed Fixed a validator bug preventing core22/core24 snaps without a `base` key from using `snapcraftctl`. Merged as #6499, resolving #6336.
73%
#6048 docs: correct 'craftctl' to 'snapctl' in config doc snapcraft merged Merged #6048 via 2d3bec6c7, correcting the config guide from craftctl to snapctl. The intersphinx link update was intentionally deferred to separate work.
71%