← Back to issue list

The `--shell` action for `pack` is broken

View original Github issue

Metadata

Project
charmcraft
Number
#650
Type
issue
State
closed
Author
facundobatista
Labels
—
Created
Updated
Closed

Current evaluation

Reporter facundobatista found `pack --shell` broken after output moved to the Emitter; fixed by PR #740 (branch pause-emitter-when-asking-for-input, merge commit e70ed86f), merged2022-04-28.

Resolution: closed resolved

Reason: Maintainer closed the issue citing PR #740 as the fix. That PR added commits 9acf2caa "Pause the Emitter when asking the user for input." and 35409faf "Pause the emitter while launching the user-requested shell.", merged as e70ed86f from branch pause-emitter-when-asking-for-input.

Evaluated by: mr-cal-bot

No scores available.

Issue body

This is after switching to output managed by Emitter.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Reporter facundobatista found `pack --shell` broken after output moved to the Emitter; fixed by PR #740 (branch pause-emitter-when-asking-for-input, merge commit e70ed86f), merged2022-04-28.
stealth/space-bunny-alpha — closed resolved Restored `pack --shell` by pausing Emitter output while prompting for user input; merged in canonical/charmcraft#740.
qwen3.6-35b-a3b-mtp-q6 — — The broken --shell action for pack following the switch to Emitter-managed output was resolved in pull request #740.
qwen/qwen3.6-35b-a3b — — Resolved the broken --shell action for pack, caused by a switch to Emitter-managed output, via pull request #740. Issue closed.
qwen3.6-35b-a3b-mtp-q6 — — Fixed the broken --shell action for pack after the Emitter output switch via pull request #740.

Update history

No update history recorded yet.

Related work

  • Likely Fixed By: canonical/charmcraft#740 (confidence 98%)

    PR 'Pause the Emitter when asking the user for input (CRAFT-952)' was merged (e70ed86f) and is explicitly cited in the issue's closing comment.

  • Related To: charmcraft#740@e70ed86f (confidence 85%)

    Merge commit contains the two fix commits 9acf2caa and 35409faf that suppress Emitter output while prompting/launching the shell.

Related issues

Issue Project State Summary Similarity
#419 Support `--shell` and `--shell-after` for the `pack` command craft-application closed Added --shell and --shell-after support to pack via merged PR #436 (commit ab55ec81); #419 was closed as a duplicate of #430, which the PR also fixed.
75%
#430 Properly support `--shell`, `--shell-after` and `--debug` on `pack` craft-application closed Fixed by merged PR #436 (ab55ec81), adding --shell and --shell-after to pack and applying --debug when packing fails.
73%