← Back to issue list

packages: run snap operations silently

View original Github issue

Metadata

Project
craft-parts
Number
#228
Type
pull request
State
merged
Author
cmatsuoka
Labels
—
Created
Updated
Closed

Current evaluation

Merged #228 (commit 17522579; implementation 29ef1aa2), silencing snap subprocesses to stop stdout leaking into craft-cli lifecycle execution and fix #218.

Resolution: closed resolved

Reason: PR #228 was merged via commit 17522579, with implementation commit 29ef1aa2 redirecting snap subprocess output to DEVNULL. It explicitly fixes issue #218.

Evaluated by: mr-cal-bot

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
stealth/space-bunny-alpha — closed resolved Merged #228 (commit 17522579; implementation 29ef1aa2), silencing snap subprocesses to stop stdout leaking into craft-cli lifecycle execution and fix #218.
stealth/space-bunny-alpha — closed resolved Merged PR #228 via commit 29ef1aa2, making snap subprocesses silent to prevent stdout leakage into craft-cli lifecycle execution; fixes #218.
stealth/space-bunny-alpha — closed resolved PR #228 was merged as 17522579, silencing snap subprocess output to prevent stdout conflicts with craft-cli. It fixes #218.
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 work

  • Related To: canonical/craft-parts#218 (confidence 100%)

    The merged pull request explicitly states that it fixes stdout leakage reported in issue #218.

Related issues

Issue Project State Summary Similarity
#3317 plugin handler: properly handle snapcraftctl errors snapcraft merged Merged PR #3317 in commit 9c622ecda, making snapcraftctl exit nonzero on build/pull failures and stop override scripts after parent operation errors.
70%