fix(remote-build): don't fail with certain archs
Metadata
Current evaluation
Merged into the hotfix branch as commit 53ec55cef, fixing architecture-specific remote-build failures from #4780, with companion support in craft-application#343.
Resolution: closed resolved
Reason: PR #4793 was approved and merged after being retargeted to the hotfix branch. Commit 53ec55cef implements the fix for the initially reported cases in #4780.
Evaluated by: mr-cal-bot
No scores available.
Issue body
This fixes the initial reported bug in #4780. However, I haven't tested some other use cases listed in that bug report.
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `tox run -m lint`?
- [x] 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 into the hotfix branch as commit 53ec55cef, fixing architecture-specific remote-build failures from #4780, with companion support in craft-application#343. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged into the hotfix branch as 53ec55cef, fixing architecture-specific remote-build failures from #4780; companion support landed in canonical/craft-application#343. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged to the hotfix branch in #4793 (commit 53ec55cef), fixing the initial architecture-specific remote-build failure from #4780. Companion support landed in craft-application#343. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a fix for remote-build architecture failures. The changes were retargeted to the hotfix branch per reviewer request before approval and merge. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged after retargeting to the hotfix branch. Resolves remote-build failures on specific architectures reported in issue #4780. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/snapcraft#4780
(confidence 100%)
Reported the architecture-specific remote-build failure fixed by this PR.
-
Related To:
canonical/craft-application#343
(confidence 95%)
Provided companion support for the remote-build fix.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #4780 remote build: fails with some architecture definitions | snapcraft | closed | Remote-build architecture handling was fixed by merged PR canonical/snapcraft#4793 (commit 53ec55cef), with companion support in canonical/craft-application#343. | |
| #4495 remote build: odd errors for certain `architectures:` setups | snapcraft | closed | Remote-build architecture handling was improved; #4994 (5cd480725) fixed CLI validation and #5005 (ad3b54bbd) rejected unsupported duplicate build-on artifacts. | |
| #5265 fix(remotebuild): don't fail when cross-compiling | snapcraft | merged | Merged PR #5265 fixed remote cross-compilation by sending build-on architectures to Launchpad; commit f94da8f13 closed #4996. | |
| #4995 remote build: error if multiple artefacts can be built on the same architecture | snapcraft | closed | Fixed by merged PR #5005 (commit ad3b54bbd): remote builds now fail early when multiple artifacts share a build-on architecture. | |
| #2774 remote-build: architecture handling | snapcraft | merged | Merged PR #2774 via commit 8300281d6, cleaning up and fixing remote-build architecture handling. Missing CLI unit tests were acknowledged as follow-up work. | |
| #5819 fix(legacy): accept --build-for on long-form architecture remote build | snapcraft | closed | PR #5819 was closed as superseded by merged PR #6090 (commit 527ff6d2c), which improved the legacy remote-builder’s --build-for error handling instead. | |
| #5052 fix: catch remote build errors | snapcraft | merged | Merged in #5052 (commit 741e36eb9), updating craft-application to 4.2.4 so remote-build errors are caught as CraftErrors and include documentation links; fixes #4908. | |
| #4392 fix(project): improve error message for unsupported architectures | snapcraft | merged | Merged PR #4392 into hotfix/7.5, improving unsupported-architecture errors and resolving #4214 (commit 0330bc36e). | |
| #5005 fix(remotebuild): error early for multiple artifacts per build-on | snapcraft | merged | Merged PR #5005 as commit ad3b54bbd, adding preflight validation to reject remote builds with multiple artifacts on the same build-on architecture; fixes #4995. |