← 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

PR #17 merged and removed Chisel from rockcraft.yaml, aligning the Snapcraft 7 rock with Snapcraft's capabilities; PR #18 later restored Chisel for Snapcraft 8.

Resolution: closed resolved

Reason: snapcraft-rocks#17 was merged after approval, removing Chisel because Snapcraft 7 does not support it. PR #18 separately restored Chisel for the Snapcraft 8 suite.

Evaluated by: mr-cal-bot

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
stealth/space-bunny-alpha — closed resolved PR #17 merged and removed Chisel from rockcraft.yaml, aligning the Snapcraft 7 rock with Snapcraft's capabilities; PR #18 later restored Chisel for Snapcraft 8.
stealth/space-bunny-alpha — closed resolved Merged canonical/snapcraft-rocks#17, removing Chisel from rockcraft.yaml so the Snapcraft rock matches regular Snapcraft 7, which does not support Chisel.
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 work

  • Related To: canonical/snapcraft-rocks#18 (confidence 99%)

    Merged follow-up restored Chisel and its spread test specifically for Snapcraft 8.

Related issues

Issue Project State Summary Similarity
#464 Make chisel a build snap rockcraft closed Closed unmerged and superseded by canonical/rockcraft#758, which carried forward removing Chisel packaging after craft-parts gained equivalent support.
81%
#758 remove chisel part from Rockcraft's snap rockcraft open Removes Chisel packaging from Rockcraft’s snap after craft-parts gained equivalent support. The prerequisite merged, but this 413-day-idle PR has no review or CI results.
81%
#84 Add chisel to rockcraft rockcraft merged Merged PR #84 as commit ddf018ce, adding Chisel to Rockcraft’s snap and enabling Chisel slices in stage-packages.
80%
#18 build: readd chisel for snapcraft 8 snapcraft-rocks merged PR #18 merged, restoring Chisel and its Snapcraft 8.x-only spread test while bumping Snapcraft to 8.0.2.
79%
#230 snap: update chisel to latest main rockcraft merged Merged PR #230 as 32aed8e7, updating Rockcraft’s snap to use the latest Chisel revision from main.
77%
#4522 chore(deps): bump version of Chisel snapcraft merged Merged PR #4522, updating Snapcraft's staged Chisel dependency to v0.8.1 for compatibility with upcoming Chisel and chisel-releases provisioning changes.
72%
#449 chore(chisel): update Chisel to v0.8.1 rockcraft merged Merged #449 as 99443eed, updating Rockcraft’s Chisel dependency to v0.8.1 for a longer migration grace period before an incompatible release.
72%
#1267 docs: Remove Chisel-related duplicated content rockcraft merged Merged as commit 888acff0: removed outdated, duplicated Chisel how-to docs from Rockcraft, deferring to Chisel's own docs; explanation page now covers Chisel's relation to Rockcraft, and a flaky custom-slice Spread test was dropped.
72%
#792 ci: install chisel as a snap craft-parts closed Closed unmerged after inactivity; equivalent Chisel-snap CI and home-fixture changes were merged in craft-parts#793, fixing craft-parts#746.
71%
#587 docs: add rockcraft chisel notes craft-parts merged Docs PR approved and merged, landing as canonical/rockcraft#98 (commit 3a87e663): added Chisel explanations and a Rockcraft Parts reference, fulfilling canonical/rockcraft#89.
70%