fix: parse build-base for snapcraftctl validation
Metadata
Current evaluation
Fixed a validator bug preventing core22/core24 snaps without a `base` key from using `snapcraftctl`. Merged as #6499, resolving #6336.
Resolution: closed resolved
Reason: The PR was merged, fixing a validator bug that incorrectly blocked core22 and core24 snaps without a `base` key from using `snapcraftctl`. Resolves #6336.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Fixes a bug where core22 and core24 snaps without a `base` key wouldn't be allowed to use `snapcraftctl`, due to a simple validator bug.
Fixes #6336
(SNAPCRAFT-1371)
---
- [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`.
- [x] I've added or updated any relevant documentation.
- [x] In documents I changed, I [added a meta description](https://canonical-starflow.readthedocs-hosted.com/how-to/add-a-page-meta-description/) if one was missing.
- [x] I've updated the relevant release notes.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | closed resolved | Fixed a validator bug preventing core22/core24 snaps without a `base` key from using `snapcraftctl`. Merged as #6499, resolving #6336. |
Update history
| Date | Change |
|---|---|
| merged | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| created |
Related work
-
Related To:
snapcraft#6336
(confidence 100%)
Original issue reporting the validator bug that this PR resolves.
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. | |
| #6193 fix: snapcraft try error shows effective base | snapcraft | merged | Merged PR #6193 fixed `snapcraft try` to report the project's effective base via `get_effective_base()`, with core24 and core26 test coverage, in commit bcae1c3ed. | |
| #4118 init: use core22 base for `snapcraft init` | snapcraft | merged | Merged PR #4118 (mr-cal, CRAFT-1670): snapcraft init now scaffolds core22 as the default base instead of core20, with the command moved from snapcraft_legacy/ to snapcraft/ and a spread test added. Squash-merged as 4afc9e4c9. | |
| #4850 fix: allow coreXX-desktop bases | snapcraft | merged | Merged PR #4850 in commit 61efc790d, adding support for core22-desktop and core24-desktop bases and fixing the Snapcraft 8.x build regression. | |
| #4580 fix: do not propagate build-base to snap.yaml | snapcraft | merged | Merged #4580 (d4e948f33), preventing build-base from being propagated into generated snap.yaml files. | |
| #1819290 Support for "base: core" | snapcraft (launchpad) | closed | Implemented by merged snapcore/snapcraft#2499 (commit e1cc2b91c), adding 'base: core' support in snapcraft.yaml and stripping it from generated snap.yaml; Launchpad base config updated by cjwatson. | |
| #5388 fix: don't fail when checking the base | snapcraft | merged | Merged PR #5388 in commit 5c43d861c, deferring base-check failures so invalid base definitions reach Snapcraft's user-friendly validation errors (SNAPCRAFT-313). | |
| #1835532 snapcraft behavior for type: base | snapcraft (launchpad) | closed | Implemented build-base support for base snaps; the proposed branch successfully built core20 using core18 before the issue was closed. | |
| #3579 snap: move base to core20 (CRAFT-509) | snapcraft | merged | Merged PR #3579 via ed2baf8bd, moving Snapcraft's own snap to the core20 base; implementation commit 18a974e01. | |
| #1976221 Missing linker base while building with core22 base | snapcraft (launchpad) | closed | Resolved by upgrading from legacy Snapcraft to Snapcraft 7.x, which correctly builds core22 snaps; no code fix or linked PR was recorded. |