← Back to issue list

build: drop chisel from rockcraft.yaml

View original Github issue

Metadata

Project
snapcraft-rocks
Number
#17
Type
pull request
State
merged
Author
tigarmo
Labels
Created
Updated
Closed

Current evaluation

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.

Suggested action:

No scores available.

Issue body

I don't recall *why* specifically I added chisel in the Rockcraft project, but the Snapcraft rock should not be *more* capable than regular Snapcraft. This means using Chisel only if provided by Snapcraft itself, and Snapcraft 7 does *not* support Chisel. When merging this to core22-8 we'll re-add the Chisel spread test, but in a v8-specific suite.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b 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.
qwen3.6-35b-a3b-mtp-q6 Merged removal of chisel from rockcraft.yaml to align the Snapcraft rock with standard Snapcraft, as version 7 lacks Chisel support. The Chisel spread test will be re-added in a v8-specific suite during the core22-8 merge.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#758 remove chisel part from Rockcraft's snap rockcraft open Removes the chisel part from Rockcraft's snap definition (13 files, +50/-17). Depends on craft-parts#904 (now merged). PR is 649 days old with no reviews, comments, or CI; chisel part still present in current snap/snapcraft.yaml.
81%
#464 Make chisel a build snap rockcraft closed Superseded by PR #758. The change to make chisel a build snap instead of embedding it in rockcraft was closed without merging after delays for chisel's stable release and project ownership transfer.
81%
#18 build: readd chisel for snapcraft 8 snapcraft-rocks merged Merged re-addition of Chisel support and spread tests for Snapcraft 8.x. The change also bumps Snapcraft to 8.0.2. Approved by two reviewers and integrated.
80%
#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.
77%
#84 Add chisel to rockcraft rockcraft merged 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.
76%
#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.
76%
#312 packages: drop release from chisel command line craft-parts merged Merged. Dropped the release argument from the chisel command line to enable native error propagation and future-proof Ubuntu version support. Approved, passed CI, and verified to correctly invoke chisel during Rockcraft builds.
75%
#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%
#746 Use chisel from snapcraft in CI instead of clone and building craft-parts closed Closed without implementation or discussion. The proposal to install chisel via snapcraft in CI was abandoned, leaving the existing clone-and-build workflow unchanged.
73%
#449 chore(chisel): update Chisel to v0.8.1 rockcraft merged Merged update to Chisel v0.8.1 to extend the migration window before an upcoming backward-incompatible release. Approved after resolving a snapcraft build-snap channel non-determinism issue. CI passed and the PR was merged.
73%