← Back to issue list

remove chisel part from Rockcraft's snap

View original Github issue

Metadata

Project
rockcraft
Number
#758
Type
pull request
State
open
Author
linostar
Labels
Created
Updated
Closed

Current evaluation

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.

Suggested action: needs review

Reason: The blocking dependency craft-parts#904 is now merged, and the chisel part still exists in snap/snapcraft.yaml, so the goal remains valid. However, the PR is 649 days old with zero reviews, comments, or CI, and the snap build system has been substantially reworked since (uv plugin, starbase, rustc), so a maintainer must assess whether the 13-file diff can be rebased or the change should be redone.

Impact: 30 Quick Win: 19.5 Staleness: 85 Complexity: 35 Confidence: 72

Issue body

- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? ----- This PR aims to stop chisel from being unnecessarily installed in the Rockcraft snap. It removes the no longer needed `chisel` part from Rockcraft's snap. This PR can only be merged when https://github.com/canonical/craft-parts/pull/904 is merged.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.8-27b
Impact: 30
Quick Win: 19.5
Staleness: 85
Complexity: 35
Confidence: 72
needs review 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.
qwen/qwen3.6-35b-a3b
Staleness: 90
Complexity: 25
Confidence: 85
close stale Removes the chisel part from Rockcraft's snap definition. PR is inactive for over a year with no reviews, comments, or CI status, and depends on an external PR.
qwen3.6-35b-a3b-mtp-q6
Staleness: 90
Complexity: 10
Confidence: 85
close stale Removes the chisel part from Rockcraft's snap to reduce unnecessary dependencies. Currently inactive for over a year and explicitly blocked by an external craft-parts PR.
qwen3.6-35b-a3b-mtp-q6
Staleness: 85
Complexity: 15
Confidence: 75
keep open Removes the chisel part from Rockcraft's snap to prevent unnecessary installation. Currently blocked on an upstream craft-parts PR and inactive for over 10 months.
qwen3.6-35b-a3b-mtp-q6
Staleness: 90
Complexity: 10
Confidence: 75
close stale Removes the chisel part from Rockcraft's snap to prevent unnecessary installation. Blocked on upstream craft-parts PR #904 and inactive for 304 days. Awaiting dependency merge to proceed with review.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/craft-parts#904 (confidence 95%)

    Original blocking dependency, now merged. Adds chisel as a conditional build snap when stage-packages contain slices, enabling this PR's removal of the unconditional chisel part.

  • Related To: canonical/snapcraft-rocks#17 (confidence 70%)

    Parallel change in the snapcraft-rocks project that also drops chisel from rockcraft.yaml, indicating a broader effort to remove unnecessary chisel dependencies.

Related issues

Issue Project State Summary Similarity
#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.
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.
75%