← Back to issue list

experimental: use binary craftctl

View original Github issue

Metadata

Project
snapcraft
Number
#6461
Type
pull request
State
open
Author
cmatsuoka
Labels
—
Created
Updated
Closed
—

Current evaluation

Maintainer's experimental +44/-9 change to snap/snapcraft.yaml to use a binary craftctl in the snap build. Open 1 day, 0 reviews, 4 unresolved review comments; CI fails Conventional Commits title and lint/files.

Suggested action: needs review

Reason: Freshly opened by a maintainer (cmatsuoka) with all functional tests green (fast, slow, minimum dependencies, snap-build) and the 'build' check still pending; the only failures are mechanical — 'policy / Check PR title for Conventional Commits' (title 'experimental: use binary craftctl' lacks a conventional type/scope) and 'lint / files' — plus 4 unresolved bot review comments, so the change still needs a maintainer/second pair of eyes on the experimental binary craftctl before landing.

Evaluated by: mr-cal-bot

Impact: 30 Quick Win: 9 Complexity: 35 Confidence: 62 Actionability: 45

Issue body

Describe your changes. --- - [ ] I've followed the [contribution guidelines](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md). - [ ] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/). - [ ] I've successfully run `make lint && make test`. - [ ] I've added or updated any relevant documentation. - [ ] In documents I changed, I [added a meta description](https://canonical-starflow.readthedocs-hosted.com/how-to/add-a-page-meta-description/) if one was missing. - [ ] I've updated the relevant release notes.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha
Impact: 30
Quick Win: 9
Complexity: 35
Confidence: 62
Actionability: 45
needs review Maintainer's experimental +44/-9 change to snap/snapcraft.yaml to use a binary craftctl in the snap build. Open 1 day, 0 reviews, 4 unresolved review comments; CI fails Conventional Commits title and lint/files.
stealth/space-bunny-alpha
Impact: 32
Quick Win: 7
Complexity: 38
Confidence: 55
Actionability: 35
needs review Maintainer cmatsuoka switches a single test helper to invoke the compiled craftctl binary instead of the Python entry point (experimental). +44/-9 lines, 1 file, 1 day old, 4 unresolved review comments.
stealth/space-bunny-alpha
Impact: 28
Quick Win: 6
Complexity: 32
Confidence: 68
Actionability: 30
keep open Experimental single-file change (+44/-9) to snap/snapcraft.yaml so the snap build pulls craftctl as a binary rather than via the normal source route. New PR by maintainer cmatsuoka, unreviewed with 4 open comments.
qwen/qwen3.8-27b
Impact: 35
Quick Win: 7
Complexity: 35
Confidence: 55
Actionability: 30
keep open Experimental change to use a binary craftctl in the snap build (snap/snapcraft.yaml), +44/-9 in 1 file. Failing CI on Conventional Commits title check and lint/files; 4 unresolved review comments; build check pending.

Update history

Date Change
created

Related work

  • Related To: snapcraft#5983 (confidence 55%)

    Merged feat!: error on snapcraftctl for core26 (eaadefa01) is the recent work constraining the Python craftctl/snapcraftctl shim semantics that this binary replacement would change.

Related issues

Issue Project State Summary Similarity
#1091 docs: revamp craftctl how-to craft-parts merged Merged PR #1091 as 44aa6508, revamping the craftctl how-to with upstream Snapcraft guidance and adding a 2.10.0 changelog entry.
72%