feat: add build-slice support, top-level build-slice key
Metadata
Current evaluation
Merged #1188, adding AppMetadata.enable_build_slices and top-level build-slices support; empty or null values are now a no-op.
Resolution: closed resolved
Reason: PR #1188 was approved and merged with build-slice support, documentation, and tests. A reported missing-Chisel failure was addressed separately by canonical/craft-parts#1737.
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 an `AppMetadata.enable_build_slices` setting.
- Adds a top-level `build-slices` key.
(CRAFT-5307)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #1188, adding AppMetadata.enable_build_slices and top-level build-slices support; empty or null values are now a no-op. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged canonical/craft-parts#1725, adding AppMetadata.enable_build_slices and a top-level build-slices key; empty or null values were finalized as a no-op. | |
| stealth/space-bunny-alpha |
Impact:
53
Quick Win:
15
Complexity:
53
Confidence:
87
Actionability:
62
|
needs review | Adds AppMetadata.enable_build_slices and top-level build-slices configuration. Approved with all CI green, but five review comments remain and witchcraft still fails locally when chisel is missing. | |
| qwen/qwen3.6-35b-a3b |
Impact:
55
Quick Win:
18
Complexity:
45
Confidence:
70
Actionability:
60
|
needs review | Adds AppMetadata.enable_build_slices setting and top-level build-slices key. New PR (0 days old), CI passing, 1 unresolved review comment, awaiting maintainer review. |
Update history
| Date | Change |
|---|---|
| closed | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| updated | |
| created |
Related work
-
Related To:
canonical/craft-parts#1737
(confidence 98%)
Adds conditional Chisel installation for parts declaring build-slices and fixes the failure reported during PR testing.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1195 Conditionally add build-slices to the json schema | craft-application | open | Triaged enhancement to conditionally expose the existing top-level build-slices field in JSON schema when AppMetadata.enable_build_slices is enabled; it remains unimplemented. | |
| #1758 feat: track build-slices in part state | craft-parts | merged | Merged PR #1758 adds build-slices to BuildState properties_of_interest so the build step invalidates when build-slices changes; updates tests. |