feat!: error on snapcraftctl
Metadata
Current evaluation
Merged #5983 as eaadefa01: snapcraftctl now errors in core26+ override scripts, and related references were renamed to craftctl.
Resolution: closed resolved
Reason: PR #5983 was approved and merged as commit eaadefa01, implementing the core26+ error and associated reference updates.
Evaluated by: mr-cal-bot
No scores available.
Issue body
This PR raises an error when `snapcraftctl` is used in an `override` script for core26+. It also renames references to snapcraftctl throughout the codebase.
(SNAPCRAFT-1226)
---
- [x] I've followed the [contribution guidelines](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md).
- [x] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/).
- [x] I've successfully run `make lint && make test`.
- [ ] I've added or updated any relevant documentation.
- [ ] I've updated the relevant release notes.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #5983 as eaadefa01: snapcraftctl now errors in core26+ override scripts, and related references were renamed to craftctl. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #5983 as eaadefa01: core26+ override scripts now reject snapcraftctl in favor of craftctl; core22/core24 compatibility remains. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #5983 as eaadefa01, making snapcraftctl fail in core26+ override scripts and renaming related internal references. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5983 as eaadefa01: snapcraftctl use now errors in core26+ override scripts, while related internal references were renamed. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged. Introduces a breaking change that errors on snapcraftctl usage in core26+ override scripts and renames all internal references. Legacy binaries are deferred to a follow-up PR, with core22/24 example updates planned for a future release. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Squash-merged. Raises an error when snapcraftctl is used in override scripts for core26+ and renames all references throughout the codebase. Related binary cleanup was handled in #5754, with future example code updates planned. |
Update history
No update history recorded yet.
Related work
-
Related To:
snapcraft#5754
(confidence 95%)
The maintainer linked #5754 while discussing wrapper cleanup; that separate removal was not part of this merge.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #3751 craftctl: add snapcraftctl compatibility wrapper | snapcraft | merged | Merged #3751 added a snapcraftctl compatibility wrapper delegating to craftctl, prompted core22 users to migrate, and restored execution of existing spread tests. | |
| #3317 plugin handler: properly handle snapcraftctl errors | snapcraft | merged | Merged PR #3317 in commit 9c622ecda, making snapcraftctl exit nonzero on build/pull failures and stop override scripts after parent operation errors. | |
| #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. | |
| #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. | |
| #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. | |
| #5782 Error when validating the model if a `snapcraftctl` command is used | snapcraft | open | snapcraftctl is now rejected during project-model validation for core26 override scripts by PR #5983; despite the issue remaining open, the requested behavior has shipped. |