feat(legacy): add a warning for the architecture shorthand on core20
Metadata
Current evaluation
Merged PR #5818 (commit f0a732be3), adding a core20 architecture-shorthand warning; released in Snapcraft 8.13.0 for #5789.
Resolution: closed resolved
Reason: The merged change closes #5789 by warning about unexpected legacy core20 shorthand behavior without changing it. Snapcraft 8.13.0 release notes list #5789 as resolved.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `make lint`?
- [x] Have you successfully run `make test`?
---
Adds a warning message when using the core20 architecture shorthand. This is because it behaves unexpectedly when compared to core22/core24, which causes confusion. See https://github.com/canonical/snapcraft/issues/5789#issuecomment-3386621445 for more context.
Closes #5789.
SNAPCRAFT-1227.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5818 (commit f0a732be3), adding a core20 architecture-shorthand warning; released in Snapcraft 8.13.0 for #5789. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5818 (commit f0a732be3) adds a warning when using the core20 architecture shorthand, closing issue #5789 without changing legacy behaviour. All test/lint CI jobs passed. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #5818 (f0a732be3), adding a legacy core20 architecture-shorthand warning to address #5789 without changing the existing behavior. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #5818 (f0a732be3) adds a legacy warning for core20 architecture shorthand, clarifying its unexpected behavior versus core22/core24 and closing #5789. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged. Adds a warning for the core20 architecture shorthand to prevent confusion with core22 and core24 behavior. Approved by two reviewers, passed CI checks, and closed issue #5789. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a change adding a warning for the core20 architecture shorthand to clarify unexpected behavior compared to core22/core24. Closes #5789. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
canonical/snapcraft#5789
(confidence 100%)
Merged PR #5818 explicitly closes this issue.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #5790 fix: resolve core20 architecture shorthand multi-snap naming issue | snapcraft | closed | External PR to fix core20 shorthand architecture producing multi-arch snaps was declined by maintainer (bepri) as better fixed internally; superseded by merged PR #5818 (commit f0a732be3) adding a warning, shipped in 8.13.0. | |
| #5789 `core20` behaviour with short-hand architecture | snapcraft | closed | Core20 shorthand packing was retained as legacy behavior: PR #5790 was closed unmerged, while PR #5818 (f0a732be3) added a warning in 8.13.0; remote-build handling moved to #5817. | |
| #4436 docs(architectures): remove shorthand notation for core22 architectures | snapcraft | merged | Merged PR #4436 in commit 0e9ea29b7, removing core22 architecture shorthand from the docs because Launchpad does not support it; Snapcraft still accepts the syntax. | |
| #5427 Add a deprecation notice for building core22 snaps with the legacy remote builder | snapcraft | closed | Added the core22 legacy remote-builder deprecation warning in Snapcraft 8.10 via merged PR #5523 (commit 1d8e87486). | |
| #2009520 core18/20: inconsistent architecture parsing | snapcraft (launchpad) | closed | Superseded by snapcore/snapcraft#4340 and #4341. Core22+ overlapping build-on handling merged in #3821 (4876b53); core20 support was declined. |