regression: remote-build doesn't accept comma-seperated architectures
Metadata
Current evaluation
Comma-separated remote-build architectures were fixed by merged PR #4543 (b8eed87c0); a later regression was superseded by #4990 and fixed by merged PR #5028 (189edf429).
Resolution: closed resolved
Reason: The reported regression was fixed by merged PR #4543 at commit b8eed87c0. After recurring in 8.2.7, it was tracked in #4990 and fixed again by merged PR #5028 at 189edf429.
Evaluated by: mr-cal-bot
No scores available.
Issue body
### Bug Description
In snapcraft 8.0.1, `snapcraft remote-build --architectures=amd64,arm64` fails to split a list of architectures.
[This logic](https://github.com/snapcore/snapcraft/blob/ffbdabf53de21606d9d12e15f16d4bca5e4423a5/snapcraft_legacy/cli/remote.py#L267) needs to be added to the remote-build cli handler.
### To Reproduce
`snapcraft remote-build --architectures=amd64,arm64`
### Environment
n/a
### snapcraft.yaml
```shell
n/a
```
### Relevant log output
```shell
2024-01-10 10:21:08.174 remote-build error: Architecture not supported by the remote builder.
The following architectures are not supported by the remote builder: ['amd64,arm64'].
Please remove them from the architecture list and try again.
```
### Additional context
_No response_
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Comma-separated remote-build architectures were fixed by merged PR #4543 (b8eed87c0); a later regression was superseded by #4990 and fixed by merged PR #5028 (189edf429). | |
| stealth/space-bunny-alpha | — | closed resolved | Remote-build architecture parsing was fixed by merged PR #4543 (b8eed87c0), then restored after regression by merged PR #5028 (189edf429). | |
| stealth/space-bunny-alpha | — | closed superseded | Originally fixed by #4543 (b8eed87c0); the regression returned in 8.2.7 and was superseded by #4990, later fixed by #5028 (189edf429). | |
| qwen/qwen3.6-35b-a3b | — | — | Fixed in PR #4543, the comma-separated architecture parsing regression reappeared in Snapcraft 8.2.7. The issue was closed as superseded, with the maintainer noting the regression will be tracked in a new issue. | |
| qwen/qwen3.6-35b-a3b | — | — | Fixed in PR #4543, the regression resurfaced in Snapcraft 8.2.7. Closed as the maintainer will track the recurring architecture parsing failure in a new report. | |
| qwen/qwen3.6-35b-a3b | — | — | Fixed in PR #4543, but the regression reoccurred in Snapcraft 8.2.7. Closed as superseded pending a new tracking issue. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Fixed in PR #4543, but the regression resurfaced in Snapcraft 8.2.7. Closed and superseded by a new tracking issue for the recurring architecture parsing bug. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
snapcraft#4543
(confidence 100%)
Merged fix for comma-separated remote-build architectures.
-
Likely Fixed By:
snapcraft#5028
(confidence 100%)
Merged follow-up fix restoring comma-separated architecture parsing.
-
Superseded By:
snapcraft#4990
(confidence 99%)
Replacement issue tracking the regression's return in Snapcraft 8.2.7.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #5028 fix(remotebuild): parse comma-separated architectures | snapcraft | merged | Merged PR #5028 (commit 189edf429) restored comma-separated parsing for remote-build --build-for and --platform, fixing #4990. | |
| #4543 fix(remote): parse comma-separated architectures | snapcraft | merged | Merged in snapcore/snapcraft#4543 as b8eed87c0, fixing comma-separated remote-build architectures and enforcing --build-for/--build-on exclusivity. | |
| #2042167 remote-build doesn't support shorthand architecture format | snapcraft (launchpad) | closed | Remote-build now handles shorthand architectures consistently; PR #4793 fixed the underlying bug in commit 53ec55cef. | |
| #4990 remote build: Allow comma-separated values for `--build-for` and `--platform` | snapcraft | closed | Regression in remote-build parsing of comma-separated `--build-for`/`--platform` values was restored by merged PR #5028 (commit 189edf429). Maintainer mr-cal closed the issue on 2024-09-13 pointing to #5028. | |
| #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. | |
| #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. |