packages: run snap operations silently
Metadata
Current evaluation
Merged to fix #218 by running snap subprocesses silently, preventing stdout leakage that conflicts with craft-cli during lifecycle execution. Approved by one reviewer, passed CI, and modified two files (+32/-15 lines).
Suggested action: —
No scores available.
Issue body
Snap installation leaks stdout output that conflicts with craft-cli
when executing the lifecycle from an application. Run subprocesses
silently to avoid this problem.
Fixes #218
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged to fix #218 by running snap subprocesses silently, preventing stdout leakage that conflicts with craft-cli during lifecycle execution. Approved by one reviewer, passed CI, and modified two files (+32/-15 lines). | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Resolves issue #218 by executing snap operations silently, preventing stdout leakage that conflicted with craft-cli during application lifecycle execution. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2256 local source: don't include .snapcraft directory | snapcraft | merged | Resolved LP: #1791773 by merging a change that excludes the .snapcraft directory from copying and part staleness checks. Approved, passed CI, and merged with a six-line update across two files. | |
| #2469 cli: clean up snapcraft push output | snapcraft | merged | Merged cleanup of the snapcraft push CLI output. Verbose messages were moved to debug level, filename formatting was standardized, and log coloring was adjusted. Approved by one reviewer, passed CI, and integrated into the main branch. | |
| #5689 fix: remove unnecessary snapd_snap flag (#5316) | snapcraft | merged | PR #5316 merged, removing the unnecessary snapd_snap flag across two files and deleting 14 lines. The change received three approvals, passed all CI checks, and was successfully integrated. | |
| #4 snap: introduce initial snap packaging (CRAFT-578) | rockcraft | merged | Merged initial snap packaging implementation (CRAFT-578). Approved by one reviewer, passed CI checks, and added 86 lines across three files. | |
| #161 snap: do not refresh already installed snaps | craft-parts | merged | Merged a port of snapcraft#3617 to craft-parts. The change prevents refreshing already installed snaps by removing two lines of code. Approved by one reviewer and passed CI checks. | |
| #2507 build providers: improve handling in snap logic | snapcraft | merged | Merged. Improved snap provider logic by cleaning up channel warnings, removing unnecessary sudo calls, and silencing noisy refresh hold commands. Approved, passed CI, and modified 5 files (+131/-73 lines). | |
| #2278 snap: pull early | snapcraft | merged | Merged a one-line change to execute snapcraftctl pull before snapcraftctl set-version. Approved by one reviewer, passed CI checks, and modified a single file. | |
| #3572 cli & providers: pass part names for lifecycle commands (CRAFT-481) | snapcraft | merged | Merged after resolving initial test and mypy failures. Passes part names to managed snapcraft for lifecycle commands and fixes a type annotation. Approved by two reviewers with 100% diff coverage. | |
| #5743 docs: fix service management to snapctl | snapcraft | merged | Merged documentation update correcting service management references from craftctl to snapctl. Approved by three reviewers, passed CI, squashed, and merged. Resolves #5738. | |
| #3173 cli: use snap pack instead of mksquashfs | snapcraft | merged | Merged. Replaced mksquashfs with snap pack, aligning with snapd standards and removing the --check-skeleton flag. Code refactored into snapcraft.cli. Windows support deferred to multipass. Approved and merged after passing CI. |