feat(remotebuild): add build-path to snap recipes
Metadata
Current evaluation
Merged PR #1128 as commit 90cc2c75, adding Launchpad build-path support to SnapRecipe and RemoteBuildService. It fixes #1106 and unblocks canonical/snapcraft#6287.
Resolution: closed resolved
Reason: The approved PR passed the relevant test and lint checks and was merged as commit 90cc2c75. The implementation forwards build_path through RemoteBuildService into snap recipe creation.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [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`?
- [x] Have you added an entry to the changelog (`docs/reference/changelog.rst`)?
---
Adds the [build-path](https://api.launchpad.net/devel.html#snaps-new) field to snap recipes and to the `RemoteBuildService`.
I tested this in a prototype branch in Snapcraft and it worked as-expected.
Fixes #1106
Unblocks https://github.com/canonical/snapcraft/issues/6287
(CRAFT-5244)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1128 as commit 90cc2c75, adding Launchpad build-path support to SnapRecipe and RemoteBuildService. It fixes #1106 and unblocks canonical/snapcraft#6287. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1128 as 90cc2c75, adding build-path to snap recipes and RemoteBuildService, fixing #1106 and unblocking Snapcraft#6287. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged. Adds build-path to snap recipes and RemoteBuildService, resolving #1106 and unblocking snapcraft #6287. Exposed as --project-dir in the remote-build command. Approved by two reviewers and passed CI. | |
| qwen/qwen3.6-35b-a3b |
Staleness:
10
Complexity:
20
Confidence:
85
|
needs review | Adds build-path field to snap recipes and RemoteBuildService. Currently approved by one reviewer with 3 unresolved comments and 2 failing CI checks. Requires maintainer review to address feedback and resolve CI before merge. | |
| qwen3.6-35b-a3b-mtp-q6 |
Staleness:
5
Complexity:
15
Confidence:
90
|
needs review | Adds build-path field to snap recipes and RemoteBuildService. Fresh PR by a maintainer fixing issue #1106 and unblocking a snapcraft dependency. Awaiting initial review. |
Update history
| Date | Change |
|---|---|
| updated |
Related work
-
Related To:
craft-application#1106
(confidence 100%)
The originating remote-build build_path support issue explicitly fixed by #1128.
-
Related To:
canonical/snapcraft#6287
(confidence 99%)
The PR states that its RemoteBuildService support unblocks the Snapcraft-side work.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1106 remote-build: support `build_path` for snaps | craft-application | closed | Closed as triaged after narrowing support to remote-build --project-dir; Snapcraft-side implementation remains in canonical/snapcraft#6287. | |
| #3648 cli: reintroduce remote-build and promote to snapcraft help | snapcraft | merged | Merged PR #3648 (commit 8dc1ac016), restoring remote-build and promote in Snapcraft's main CLI help for Launchpad bug #1943244. | |
| #196 feat: remote build service | craft-application | merged | Merged PR #196 in commit e53de201, adding remote build service support and porting the latest remote-build implementation from Snapcraft. | |
| #6287 remote-build: support build paths | snapcraft | open | Triaged enhancement to let remote-build select a nested snapcraft.yaml via --project-dir; upstream support is complete, but Snapcraft implementation and spread coverage remain. | |
| #6372 feat(remote-build): add --project-dir argument to support build paths | snapcraft | open | Adds --project-dir to snapcraft remote-build (monorepo/build paths) with unit + spread tests. Maintainer testing rejected the os.chdir/sys.argv workaround in favor of build-path; remote-build and lint CI are red, no approvals, quiet 28 days. | |
| #3208 snap: set PATH for snapcraft command | snapcraft | merged | Merged #3208 in commit cd4f5d5cb, applying the build environment’s fixed PATH to the Snapcraft command for parity with Launchpad buildd and destructive-mode runs. | |
| #4892 fix(remotebuild): use legacy launchpad credentials if they exist | snapcraft | merged | Merged in snapcraft#4892 as c1c70a249, adding legacy Launchpad credential fallback, build-for: all support, and expanded remote-build tests; fixes #4889. |