← Back to issue list

fix(commands/lifecycle): Pass the correct platform and build-for

View original Github issue

Metadata

Project
craft-application
Number
#687
Type
pull request
State
merged
Author
lengau
Labels
Created
Updated
Closed

Current evaluation

Merged fix correcting platform and build-for parameter passing in lifecycle commands, resolving a "Multiple builds match the current platform" error. Approved by two reviewers and passed CI.

Suggested action:

No scores available.

Issue body

Fixes this: ```yaml platforms: platform1: build-on: [amd64] build-for: [amd64] platform2: build-on: [amd64, arm64] build-for: [arm64] ``` -> (error comes from the managed instance, not the host) ``` Multiple builds match the current platform: 'platform1' and 'platform2'. Recommended resolution: Check the "--platform" and "--build-for" parameters. ``` - [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `make lint && make test`? - [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)? ---

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged fix correcting platform and build-for parameter passing in lifecycle commands, resolving a "Multiple builds match the current platform" error. Approved by two reviewers and passed CI.
qwen3.6-35b-a3b-mtp-q6 Merged a fix correcting platform and build-for parameter passing in lifecycle commands, resolving a multiple builds match error. Authored by maintainer lengau, the change passed lint and test suites before integration.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#442 fix(platform): automatically vectorise architectures craft-application merged Merged fix allowing the Platform model to accept string or list values for build-on and build-for architectures, resolving issue #1874. Approved by two reviewers, passed CI, and modified two files.
74%
#5351 fix(remote): error early for invalid platform usage snapcraft merged Merged a fix that raises an early error when --build-for conflicts with shorthand platform entries, preventing unfriendly Launchpad failures. The change passed CI checks and received approvals, resolving issue #5270.
73%
#365 feat: accept build-for: [all] in platforms craft-application merged Merged via rebase after approval and passing CI. Enables build-for: [all] in platform definitions, restricting other platforms when used. Resolves #360.
73%
#45 fix(publish): support build-for 'all' starflow merged Merged a fix to the publish step supporting build-for 'all'. The change prevents platform override when 'all' is defined and adds a fallback to locate architecture-specific remote-build logs. Approved by two reviewers.
70%
#94 fix: generate stricter default platforms debcraft merged Merged fix for issue #52 that generates stricter default platforms equivalent to Debian's any architecture value. Approved by two reviewers, passed all CI checks, and modified 10 files.
70%