feat(remote): allow using shorthand platforms with --build-for
Metadata
Current evaluation
Merged PR #5687 (commit aa2d9ef0d), enabling --build-for with shorthand platform declarations after Launchpad removed the underlying restrictions.
Resolution: closed resolved
Reason: PR #5687 was merged with approval and passed the relevant test, lint, security, and build checks. Its merge commit removed the validation introduced by PR #5351.
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`?
---
Now that [LP#2077005](https://bugs.launchpad.net/launchpad/+bug/2077005) and[ LP#2098811](https://bugs.launchpad.net/launchpad/+bug/2098811), Snapcraft can drop the restriction about using `--build-for` when shorthand platforms are provided.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5687 (commit aa2d9ef0d), enabling --build-for with shorthand platform declarations after Launchpad removed the underlying restrictions. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5687 (commit aa2d9ef0d), enabling shorthand platforms with remote builds’ --build-for once the cited Launchpad restrictions were lifted. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a feature enabling shorthand platforms with the --build-for flag. The change removes a prior restriction now that related Launchpad bugs are resolved. Approved by three reviewers and passed CI checks before integration. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Removes the restriction on using --build-for with shorthand platforms for remote builds, resolving Launchpad bugs LP#2077005 and LP#2098811. The change passed automated tests and manual verification before being merged by maintainer mr-cal. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/snapcraft#5351
(confidence 99%)
This earlier merged PR introduced the validation that #5687 removed.
-
Related To:
canonical/snapcraft#5270
(confidence 96%)
The original remote-build restriction was addressed by #5351 and subsequently removed by #5687.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #5351 fix(remote): error early for invalid platform usage | snapcraft | merged | PR #5351 merged as commit 0c224fc7d, adding an early, friendlier error when --build-for is used with shorthand project platforms instead of failing at Launchpad. Fixes #5270 (CRAFT-4186); restriction later lifted by #5687. | |
| #6090 feat(remote-build/legacy): improve error when building with --build-for | snapcraft | merged | Merged PR #6090 improved legacy remote-builder errors for core20, clarifying that --build-for requires shorthand architectures because of Launchpad limitations. It addresses #5817 and supersedes #5819. | |
| #5216 feat(remotebuild)!: filter build-plan with --build-for, drop --platform | snapcraft | merged | Merged as commit 359a364cb (CRAFT-3277): `remote-build` now filters the build plan with `--build-for`, `--platform` was removed, and Launchpad errors are caught instead of surfacing as internal errors. Ships in snapcraft 8.7; closes #4992. |