Skip to content

Commit

Permalink
PipeCommand: pass shell arg to ExternalCommand.
Browse files Browse the repository at this point in the history
Prior to this branch, `shell=True` was passed to Popen regardless of
whether `--shell` was passed. I'm not sure why this was the case but it
wasn't a problem. Now that we're passing to ExternalCommand, this
doesn't work because ExternalCommand does it's own manipulation of cmd
when shell=False which is incompatible with a list-style cmd.
  • Loading branch information
ryneeverett committed Dec 12, 2024
1 parent 34bb6ac commit 62c6906
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion alot/commands/thread.py
Original file line number Diff line number Diff line change
Expand Up @@ -801,7 +801,7 @@ def callback(out):

await ui.apply_command(ExternalCommand(cmd,
stdin=mail,
shell=True,
shell=self.shell,
thread=self.background,
on_success=callback))

Expand Down

0 comments on commit 62c6906

Please sign in to comment.