← Back to issue list

Add chisel to rockcraft

View original Github issue

Metadata

Project
rockcraft
Number
#84
Type
pull request
State
merged
Author
tigarmo
Labels
Created
Updated
Closed

Current evaluation

Merged after approval and passing CI. Integrated chisel into rockcraft to support chisel slices in stage-packages. Temporarily pinned craft-parts to a main branch commit pending an official release. Added a spread test for the dotnet-runtime-6.0_libs slice.

Suggested action:

No scores available.

Issue body

This PR lets rockcraft support chisel slices in stage-packages. Notes: - "chisel" is added directly on rockcraft's snap, just like skopeo and umoci, as it's only an "image-build-time" dependency; - We currently pin `craft-parts` to a specific commit on `main` because we don't yet have a release of craft-parts with chisel support, but that should come soon; - The added spread test uses the dotnet-runtime-6.0_libs slice, and currently overrides `prime` to create symlinks because the internal directory structure of the slice changes with new versions. This will be addressed/discussed in the future. CRAFT-1333

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged after approval and passing CI. Integrated chisel into rockcraft to support chisel slices in stage-packages. Temporarily pinned craft-parts to a main branch commit pending an official release. Added a spread test for the dotnet-runtime-6.0_libs slice.
qwen3.6-35b-a3b-mtp-q6 Merged. Added chisel support to rockcraft stage-packages by integrating it as a snap dependency. Temporarily pinned craft-parts to a main commit pending release. Included a spread test for the dotnet-runtime-6.0_libs slice. Resolves CRAFT-1333.
qwen3.6-35b-a3b-mtp-q6 Merged to integrate chisel into rockcraft, enabling chisel slice support in stage-packages. Chisel was added as a build-time dependency, craft-parts was temporarily pinned to a specific commit, and a spread test was included to validate the feature.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1267 docs: Remove Chisel-related duplicated content rockcraft merged Merged documentation update removing outdated Chisel content and duplicated guides. Updated the Chisel-Rockcraft relationship explanation and fixed a custom slice installation placeholder. Addressed feedback by removing a flaky Spread test. Approved and merged despite minor CI warnings.
77%
#17 build: drop chisel from rockcraft.yaml snapcraft-rocks merged Merged after removing chisel from rockcraft.yaml to align the Snapcraft rock with standard Snapcraft capabilities, as Snapcraft 7 lacks support. Approved by two reviewers, passed CI, and modified three files. A v8-specific test suite will later restore the Chisel spread test.
76%
#904 feat: add chisel as a build snap when stage-packages contain slices craft-parts merged Merged. Conditionally installs chisel as a build snap only when stage-packages contain slices and no part already declares it, preventing unnecessary installation in the Rockcraft snap. Approved by two reviewers and passed CI.
75%
#564 fix(tests): don't use chisel slices in 'devel' bases rockcraft merged Merged a fix removing chisel slices from 'devel' tests. Chisel releases lag Ubuntu development, leaving newer versions unavailable. This enables Rockcraft execution testing without missing slices. Approved by two reviewers, changing two files.
74%
#587 docs: add rockcraft chisel notes craft-parts merged Merged documentation update adding rockcraft chisel notes. Approved by two reviewers, the maintainer added nine lines to one file. Changes were integrated into the repository without CI checks or additional comments.
74%
#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.
74%
#98 Chisel documentation rockcraft merged Merged after approval and passing CI. Updated the ReadTheDocs template, corrected styling, and added a Chisel explanation with parts reference to the Rockcraft documentation.
73%
#649 Install chisel if parts use slices craft-parts closed Automatically installs chisel when parts declare slices in stage-packages. Resolved and merged via pull request #904, centralizing the tool management within craft-parts.
73%
#155 spread: fix project with dotnet chisel slices rockcraft merged Merged to remove an obsolete prime override and symlink in the spread configuration, as dotnet packages no longer create version-specific subdirectories. The change resolved CI failures blocking other pull requests.
71%
#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.
71%