fix: accept a 'pebble' part if its the same as ours
Metadata
Current evaluation
Merged fix resolving a duplicate pebble part conflict during expand-extensions packing. Rockcraft now accepts an existing pebble part if its contents match the default, resolving issue #749. Approved by two reviewers and passed CI.
Suggested action: —
No scores available.
Issue body
The use-case that this enables is packing the output of 'expand-extensions'. Previously, this would fail because the expanded yaml contains the 'pebble' part that Rockcraft adds, and then Rockcraft itself would complain that the project already has a part called 'pebble'.
This commit makes Rockcraft accept an existing part called 'pebble' if the contents of said part are the same as the one we would add.
Fixes #749
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged fix resolving a duplicate pebble part conflict during expand-extensions packing. Rockcraft now accepts an existing pebble part if its contents match the default, resolving issue #749. Approved by two reviewers and passed CI. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged fix enabling Rockcraft to accept an existing 'pebble' part if its configuration matches the expected output. This resolves part name conflicts during 'expand-extensions' packaging and closes issue #749. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged fix resolving duplicate pebble part conflicts when packing expand-extensions output. The change allows Rockcraft to accept an existing pebble part if its configuration matches the default, closing issue #749. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #310 docs: fix url of rockcraft format | rockcraft | merged | Merged a documentation fix correcting a broken URL in convert-to-pebble-layer.rst, resolving a linkcheck failure. Approved by two reviewers, passed CI, and modified one file. | |
| #244 project: constrain ROCK name to pebble's rules | rockcraft | merged | Merged changes constraining ROCK names to Pebble's rules: lowercase letters, numbers, and hyphens only. Approved by two reviewers, the update modifies five files to enforce consistent naming across layer files. | |
| #59 Install pebble via implicit part | rockcraft | merged | Merged. Adds an implicit pebble part to install Pebble as the ROCK entrypoint using the standard Go plugin. CI issues were resolved, and the code was updated to block user overrides, ensuring spec compliance. Approved and merged. |