fix: allow coreXX-desktop bases
Metadata
Current evaluation
Merged PR #4850 in commit 61efc790d, adding support for core22-desktop and core24-desktop bases and fixing the Snapcraft 8.x build regression.
Resolution: closed resolved
Reason: PR #4850 was approved and merged as commit 61efc790d. It resolved issue #4818 by allowing coreXX-desktop bases in Snapcraft 8.x.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Fixes #4818
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `tox run -m lint`?
- [ ] Have you successfully run `tox run -e test-py310`? (supported versions: `py39`, `py310`, `py311`, `py312`)
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #4850 in commit 61efc790d, adding support for core22-desktop and core24-desktop bases and fixing the Snapcraft 8.x build regression. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #4850 in commit 61efc790d, allowing core22-desktop and core24-desktop bases and fixing the 8.x build regression in #4818. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged to allow coreXX-desktop bases, resolving #4818. Approved by two reviewers, passed CI checks, and integrated with minimal code changes. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to resolve issue #4818 by enabling support for coreXX-desktop bases. The fix was integrated into the main branch. |
Update history
No update history recorded yet.
Related work
-
Related To:
snapcraft#4818
(confidence 100%)
The merged PR explicitly fixes the desktop-base build regression reported in this issue.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #4186 snap: migrate to core22 | snapcraft | merged | PR #4186 merged, moving the Snapcraft snap's own build environment to base core22 (8 files, +71/-70), landing as commit 60033305b and closing the core22 migration issue #4166. | |
| #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. | |
| #6338 docs: update supported bases | snapcraft | merged | Merged PR #6338 (commit 2a6aadac6), updating the docs to identify Snapcraft 9 as preferred for core22, core24, and core26. | |
| #6499 fix: parse build-base for snapcraftctl validation | snapcraft | merged | Fixed a validator bug preventing core22/core24 snaps without a `base` key from using `snapcraftctl`. Merged as #6499, resolving #6336. | |
| #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. | |
| #2761 Core20 base | snapcraft | merged | Core20 base support was merged via #3579, moving Snapcraft to core20 in commit 18a974e01 (merge ed2baf8bd). | |
| #6150 docs: use core26 as the default base | snapcraft | merged | Merged #6150 in commit 9b6d7fa9a, documenting core26 as the default base, publishing Snapcraft 9 release notes, and standardizing core/base terminology. | |
| #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. | |
| #3583 build providers: snapcraft's new base is core20 (CRAFT-544) | snapcraft | merged | Merged PR #3583 via 2b2d7dbcf, replacing build providers' injected core18 base with core20 to match Snapcraft's new base. |