docs(architectures): reword how to section
Metadata
Current evaluation
Merged #4362 (commit c6a4cf7a1), clarifying architecture-build docs and removing cross-compiling wording because Snapcraft does not automatically cross-compile.
Resolution: closed resolved
Reason: PR #4362 was merged into main as c6a4cf7a1. Its documentation change clarified building for another architecture without implying automatic cross-compilation.
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`?
-----
Use more accurate wording for building for a different architecture. Remove 'cross-compiling' because snapcraft doesn't automatically cross-compile.
From @cmatsuoka's suggestion in https://github.com/snapcore/snapcraft/issues/4336#issuecomment-1712592331
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #4362 (commit c6a4cf7a1), clarifying architecture-build docs and removing cross-compiling wording because Snapcraft does not automatically cross-compile. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in #4362 (c6a4cf7a1), reworded architecture build instructions to remove “cross-compiling” and clarify that Snapcraft does not automatically cross-compile. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a documentation update that reworded the architecture section to remove the term cross-compiling, clarifying snapcraft does not auto-cross-compile. Approved by two reviewers, passed CI, and merged. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged documentation updates to accurately describe building for different architectures, removing misleading cross-compiling terminology. Changes were successfully integrated into main after passing CI checks. |
Update history
No update history recorded yet.
Related work
-
Related To:
snapcore/snapcraft#4336
(confidence 99%)
The wording was updated from the maintainer suggestion in issue #4336.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #4349 docs(architectures): add how-to, reference, and explanation | snapcraft | merged | Merged PR #4349 as commit c8e9b2e42, adding architecture how-to, reference, and explanation guides and resolving #4336. | |
| #411 docs: refer to the working directory | craft-parts | merged | Merged docs cleanup in #411 (commit cf66c5c3), replacing overly Snapcraft-specific wording with references to the working directory. | |
| #5359 docs: migrate build environment reference page | snapcraft | merged | Merged PR #5359 via commit 1678a1ea9, migrating the legacy build-options page's build environment reference into Snapcraft's in-repo documentation. | |
| #4336 Update architectures documentation | snapcraft | closed | Closed without a linked resolution after documentation-planning comments. Later platform documentation was rewritten in canonical/snapcraft#6069 (commit 88b36a7d6). | |
| #5586 docs: improve and move setup guides | snapcraft | merged | Merged PR #5586 improved and relocated the Snapcraft setup guides based on internal feedback, landing as commit dacb9c702. | |
| #6107 docs: improve the contribution guides more | snapcraft | merged | Merged documentation improvements in snapcraft#6107 via commit aced3d1aa, refining the commit workflow, testing guidance, and link accessibility. |