feat(remotebuild): add build-path to snap recipes
Metadata
Current evaluation
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.
Suggested action: —
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 |
|---|---|---|---|---|
| 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 issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #6372 feat(remote-build): add --project-dir argument to support build paths | snapcraft | open | Adds --project-dir argument to snapcraft remote-build for monorepo support. Under active review with maintainer suggesting alternative approaches using build-path or upstream monorepo feature. | |
| #6287 remote-build: support build paths | snapcraft | open | Enhancement to add --project-dir support for remote-build command, allowing users to specify build paths when multiple snapcraft.yaml files exist. Issue is triaged and assigned to a contributor after upstream dependency was resolved. | |
| #1106 remote-build: support `build_path` for snaps | craft-application | closed | Accepted and closed. The build_path enhancement for snaps is scoped to the remote-build subcommand for now, with plans to expand to other lifecycle commands later. | |
| #196 feat: remote build service | craft-application | merged | Merged the remote build service feature, migrating code from snapcraft. Approved by two reviewers, passed CI checks, and introduced 2316 lines across 26 files. | |
| #4353 feat(remote-build): check SNAPCRAFT_REMOTE_BUILD_STRATEGY envvar | snapcraft | merged | Merged implementation of SNAPCRAFT_REMOTE_BUILD_STRATEGY environment variable to control remote build fallback for core22 and older bases. Approved by two reviewers, passed CI, and achieved 100% diff coverage. Resolves #4319. | |
| #2500 cli: add remote build | snapcraft | merged | Merged. Adds remote build capability to Launchpad, dispatching source trees for remote compilation and retrieving resulting snap packages or logs. Approved by reviewers, passed CI, and integrated despite a minor test coverage decrease. | |
| #4395 feat(remote-build): new command logic | snapcraft | merged | Merged into main. Introduces a RemoteBuild class to orchestrate remote builds, adds local project directory caching, and implements post-build cache cleanup. Resolves #4324. Approved by three reviewers with passing CI and 97.88% diff coverage. | |
| #3648 cli: reintroduce remote-build and promote to snapcraft help | snapcraft | merged | Merged the reintroduction of the remote-build CLI command and promoted it to snapcraft help. Approved by one reviewer, passed CI checks, and resolved Launchpad bug #1943244 via a single-file, four-line deletion. | |
| #3208 snap: set PATH for snapcraft command | snapcraft | merged | Merged to configure the PATH for the snapcraft command in the build environment, ensuring parity with launchpad buildd and destructive-mode instances. Approved, passed CI, and modified three files. | |
| #373 feat(remotebuild): add property for launchpad credentials filepath | craft-application | merged | Merged after approval and passing CI. Added a credentials_filepath property to remotebuild, enabling applications to specify a custom credentials file. Unblocks related snapcraft dependencies. |