← Back to issue list

cli: disable --target-arch for multipass/lxd

View original Github issue

Metadata

Project
snapcraft
Number
#3152
Type
pull request
State
merged
Author
cjp256
Labels
—
Created
Updated
Closed

Current evaluation

PR #3152 merged as commit 2dd5e5d1e, hiding/disabling the --target-arch option for Multipass and LXD builds because it had no effect; follow-up PR #3175 later restored it with a deprecation warning.

Resolution: closed resolved

Reason: The one-line change was merged into master via commit 2dd5e5d1e ("cli: disable --target-arch for multipass/lxd (#3152)"), removing the non-functional --target-arch option for LXD/Multipass providers. Architecture targeting was later reworked, with PR #3175 restoring the flag behind a warning.

Evaluated by: mr-cal-bot

No scores available.

Issue body

It currently doesn't have any effect, so rather than imply that it is in fact working, disable it outright. We will be working toward expanding "--build-on" to target other architectures without cross-compilation that is broken in many cases (like this). Signed-off-by: Chris Patterson <chris.patterson@canonical.com> - [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `./runtests.sh static`? - [ ] Have you successfully run `./runtests.sh tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved PR #3152 merged as commit 2dd5e5d1e, hiding/disabling the --target-arch option for Multipass and LXD builds because it had no effect; follow-up PR #3175 later restored it with a deprecation warning.
stealth/space-bunny-alpha — closed resolved Merged PR #3152 disables --target-arch for Multipass and LXD because it had no effect, avoiding a misleading option while architecture support is reworked.
qwen/qwen3.6-35b-a3b — — Merged changes disabling the --target-arch CLI flag for multipass and LXD. The flag was non-functional and misleading, so it was removed to prevent confusion while future architecture targeting is developed. Approved and passed CI.
qwen3.6-35b-a3b-mtp-q6 — — Merged into master. The --target-arch CLI flag was disabled for multipass and LXD backends as it currently has no effect. This prevents misleading users while future --build-on improvements are developed. Codecov reported a negligible coverage decrease.

Update history

No update history recorded yet.

Related work

  • Related To: snapcraft#3175 (confidence 90%)

    Commit b29e100eb 'cli: restore --target-arch with warning for LXD and Multipass (#3175)' partially reverses this PR a few weeks later.

  • Related To: snapcraft#3410 (confidence 70%)

    Commit ceee8c010 'project: enable experimental target-arch support for core20' reintroduces target-arch support at the project level.

  • Related To: snapcraft#3151 (confidence 65%)

    Companion PR #3151 'cli: disable --target-arch support on core20' from the same architecture-rework effort.

Related issues

Issue Project State Summary Similarity
#3175 cli: restore --target-arch with warning for LXD and Multipass snapcraft merged Merged in #3175 (b29e100eb), restoring --target-arch while warning that architecture targeting is unsupported for LXD and Multipass.
83%
#3172 cli: restore --target-arch with warning for LXD and Multipass snapcraft closed Closed unmerged; its change landed as #3175 (commit b29e100eb), restoring --target-arch for LXD/Multipass with a warning, followed by fix #3181 (9346aa623).
82%
#1493 lxd: only pass target arch if specified explicitly snapcraft merged Merged via commit 0a7de4515, passing --target-arch to LXD only when explicitly set so builds default to the container architecture.
76%
#3151 cli: disable --target-arch support on core20 snapcraft merged Merged PR #3151 as commit 8942d89d2, disabling --target-arch for core20 because cross-compilation had limitations; architecture targeting was deferred pending better --build-on support.
72%