← Back to issue list

fix: resolve core20 architecture shorthand multi-snap naming issue

View original Github issue

Metadata

Project
snapcraft
Number
#5790
Type
pull request
State
closed
Author
Nalin-Kumar-Gupta
Labels
Created
Updated
Closed

Current evaluation

Rejected by maintainers due to unintended behavioral changes. The contributor will step back while the team implements an internal fix for the core20 architecture shorthand multi-snap naming issue.

Suggested action:

No scores available.

Issue body

- Fixes (https://github.com/canonical/snapcraft/issues/5789) - Fix shorthand architectures (e.g., [arm64, amd64]) creating 'multi' snaps instead of architecture-specific snaps - Update legacy architecture processing to find matching host architecture in shorthand form - Ensure snap metadata uses resolved build-for architecture instead of raw architectures list - Make shorthand and expanded architecture forms behave equivalently Fixes issue where core20 snaps with shorthand architecture definitions would produce foo_0.1_multi.snap instead of the expected foo_0.1_amd64.snap when building on amd64. The expanded form: architectures: - build-on: [arm64] run-on: [arm64] - build-on: [amd64] run-on: [amd64] Now behaves identically to the shorthand form: architectures: [arm64, amd64] Both forms will now build architecture-specific snaps rather than multi-arch snaps. - [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`? ---

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Rejected by maintainers due to unintended behavioral changes. The contributor will step back while the team implements an internal fix for the core20 architecture shorthand multi-snap naming issue.
qwen3.6-35b-a3b-mtp-q6 Rejected due to unintended side effects. Maintainers declined the core20 architecture shorthand fix, preferring an internal implementation and advising contributors to coordinate before submitting changes.
qwen3.6-35b-a3b-mtp-q6 Rejected and closed. Maintainers declined the merge, noting the changes introduce unintended behavioral side effects and will be resolved internally.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#5789 `core20` behaviour with short-hand architecture snapcraft closed Closed without fixing the core20 shorthand architecture bug, deemed a load-bearing behavior that cannot be changed. The related remote-build inconsistency was identified as a separate fixable issue tracked in #5817.
70%
#5588 fix!: use platform name in snap filename snapcraft merged Merged a fix updating Core24 snap filenames to use the platform name instead of the build-for architecture. This resolves a file collision bug where identical architectures caused overwrites. Approved, passed CI, and closes issue #5399.
70%