fix(deb): normalize chisel slices
Metadata
Current evaluation
Merged fix normalizing chisel slices during deb unpacking to correct absolute symlinks, resolving issue #578. Approved by two reviewers with passing CI checks.
Suggested action: —
No scores available.
Issue body
When stage debs are unpacked, we "normalize" the contents to do things like fix absolute symlinks (as absolute links would point outside the lifecycle environment). Do the same for cut chisel slices too.
Fixes #578
- [ ] 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 normalizing chisel slices during deb unpacking to correct absolute symlinks, resolving issue #578. Approved by two reviewers with passing CI checks. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix to normalize chisel slices during deb unpacking, resolving absolute symlink issues by applying stage deb normalization. Closes #578. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #592 fix(deb): normalize chisel slices | craft-parts | merged | Merged. Normalizes chisel slices during deb unpacking to fix absolute symlinks and prevent external path references. Resolves #578. Approved by two reviewers and passed CI. | |
| #279 spec: add "stage-slices" for chisel slices | craft-parts | merged | Merged the addition of a stage-slices specification to handle chisel slice naming conventions, which require underscores unlike standard .deb packages. Approved by one reviewer, modified two files, and resolved CRAFT-1329. | |
| #611 Release 1.26.1 | craft-parts | merged | Merged release 1.26.1 following approval from two reviewers and successful CI checks. The update resolves chisel slice normalization issues and addresses Sphinx warnings across five files. | |
| #127 fix: pin the commit of the bundled chisel | rockcraft | merged | Merged a one-line fix that pins the bundled chisel dependency to a specific commit. Approved by one reviewer with no unresolved comments or CI issues. | |
| #281 unpack chisel slices | craft-parts | merged | Merged implementation to unpack chisel slices via PullState routing from pull to build step. Updated CI Go version, added integration tests, and verified locally. Approved by two reviewers, +365/-66 lines across 8 files. | |
| #4568 chore(deps): bump version of Chisel (#4522) | snapcraft | merged | Merged dependency update bumping Chisel version. Approved by two reviewers, passed CI, and cherry-picked onto the feature/components branch. Single-file change with minimal modifications. | |
| #1715 refactor: put chisel call in its own module | craft-parts | merged | Merged refactoring that extracts the chisel call into a dedicated module, decoupling slice cutting from path normalization to support upcoming build-slices changes. Approved by two reviewers. | |
| #1538 feat(packages/deb): parse chisel manifests | craft-parts | merged | Squash-merged feature adding Chisel manifest parsing to the deb package handler. Filters stage-packages for Core26 by reading manifest files. Approved by two reviewers, passed CI checks, and unblocks downstream snapcraft issues. |