← Back to issue list

build: readd chisel for snapcraft 8

View original Github issue

Metadata

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

Current evaluation

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.

Suggested action:

No scores available.

Issue body

Re-add Chisel support for the Snapcraft 8.x branch. This re-adds the spread test in a new suite that only runs for Snapcraft 8 rocks. This PR has 5 commits because it includes a merge; only the last two are new changes. The first one re-adds Chisel and its spread test, and the second one bumps Snapcraft to 8.0.2

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b 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.
qwen3.6-35b-a3b-mtp-q6 Merged to re-add Chisel support and its spread test for Snapcraft 8.x. The change includes a merge commit and bumps Snapcraft to 8.0.2.

Update history

No update history recorded yet.

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.
80%
#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.
76%
#230 snap: update chisel to latest main rockcraft merged Merged update to the chisel dependency, aligning the snap package with the latest main branch. Approved by two reviewers with zero comments and a single-line change.
76%
#456 fix(spread): tests must also use newer Chisel rockcraft merged Merged. Resolves failing spread tests by switching Chisel installation from a hardcoded commit to the snap store. Addressed review feedback and re-triggered CI after a timeout.
71%
#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.
71%
#1 feat: add project for core22 + snapcraft 7 snapcraft-rocks merged Merged adds a core22 and snapcraft 7 project providing a foundational build environment for hello snaps and chisel slices. Approved by two reviewers, the change adds 116 lines to a single file.
70%
#15 feat: add Snapcraft 8.x rock snapcraft-rocks merged Merged a minimal change adding a Snapcraft 8.x rock. Approved by two reviewers and implemented by maintainer tigarmo. Accepted without comments or CI checks.
70%