← Back to issue list

architectures: add schema support

View original Github issue

Metadata

Project
snapcraft
Number
#3812
Type
pull request
State
merged
Author
mr-cal
Labels
—
Created
Updated
Closed

Current evaluation

Merged as commit 629e4ce51: adds core22 architectures schema support to snapcraft.yaml (build_to renamed to build_for), part 1 of CRAFT-1163; full feature followed in #3821 (4876b53).

Resolution: closed resolved

Reason: The PR itself was merged into main as commit 629e4ce51, adding the architectures schema definition and renaming build_to to build_for. The larger originally proposed PR #3712 was closed as outdated and split up, with the remaining core22 architecture implementation merged separately in #3821 (4876b53).

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [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 `make lint`? - [X] Have you successfully run `pytest tests/unit`? ----- Add support for defining architectures in snapcraft.yaml. Part 1 of supporting architectures for core22. Full code change is proposed [here](https://github.com/snapcore/snapcraft/pull/3712). (CRAFT-1163)

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged as commit 629e4ce51: adds core22 architectures schema support to snapcraft.yaml (build_to renamed to build_for), part 1 of CRAFT-1163; full feature followed in #3821 (4876b53).
stealth/space-bunny-alpha — closed resolved Merged PR #3812 as commit 629e4ce51, adding core22 snapcraft.yaml architectures schema support (part 1 of CRAFT-1163); renamed build_to to build_for. Follow-up work landed in #3821.
stealth/space-bunny-alpha — closed resolved Merged PR #3812 as commit 629e4ce51, adding core22 architecture schema support for snapcraft.yaml as part one of CRAFT-1163; build_to was renamed to build_for.
stealth/space-bunny-alpha — closed resolved PR #3812 merged into main, adding schema support for defining architectures in snapcraft.yaml as part one of core22 architecture support; build_to was renamed to build_for.
stealth/space-bunny-alpha — closed resolved Merged PR #3812 added snapcraft.yaml architecture schema support for core22, landing as commit 629e4ce51.
qwen/qwen3.6-35b-a3b — — Merged after approval and passing CI. Added schema support for defining architectures in snapcraft.yaml as part one for core22. Coverage increased by 0.09%, and the build_to parameter was renamed to build_for during review.
qwen3.6-35b-a3b-mtp-q6 — — Merged into main, adding schema support for defining architectures in snapcraft.yaml as part of core22 implementation. The build_to parameter was renamed to build_for. The change passed all checks and improved test coverage.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/snapcraft#3821 (confidence 88%)

    Merged follow-up (4876b53) implementing core22 architecture handling: build-on/build-for, host/build-instance selection, SNAPCRAFT_BUILD_FOR.

  • Superseded By: canonical/snapcraft#3712 (confidence 70%)

    Original full-scope proposal, closed as outdated and split; #3812 delivers part 1 (schema).

Related issues

Issue Project State Summary Similarity
#5606 schema: add build-for in architectures for core22 snapcraft merged Merged as c9822a38c via snapcraft#5606, adding build-for to the core22 architectures schema in snapcraft.json.
80%
#1685 Implement support for architecture in snapcraft.yaml snapcraft closed Architecture support for snapcraft.yaml landed in PR #2080 (commit c60ac3891), adding CI-oriented build-on/run-on lists. build.snapcraft.io adoption was tracked separately by cjwatson.
76%
#3821 meta: architectures snapcraft merged Merged core22 architecture support, including build-on/build-for handling, host and build-instance selection, and SNAPCRAFT_BUILD_FOR filtering, via commit 4876b53.
74%
#4150 meta: use build-for in snap.yaml architecture snapcraft merged Merged #4150 as 77d3454d: build-for: all now writes architectures: all to meta/snap.yaml instead of the build-on architecture, fixing CRAFT-1535.
71%
#2446 Update schema/snapcraft.yaml snapcraft closed PR #2446 edited schema/snapcraft.yaml to aid yaml-to-json converters but was closed unmerged in favour of #2448, which replaced the YAML schema with snapcraft.json (commit 2a2f798d0).
70%
#2627 synchronizing snapd supported schema to snapcraft snapcraft merged Merged in snapcore/snapcraft#2627 (5534fe3d), synchronizing Snapcraft’s schema with snapd’s supported daemon and app fields and validation patterns.
70%
#3565 schema: add environment support for hooks (CRAFT-424) snapcraft merged Merged PR snapcore/snapcraft#3565, adding CRAFT-424 hook environment support, shared schema definitions, snap metadata processing, and unit/spread tests.
70%