Skip to content

tmux new does not take tw commands

Peter Seebach openembedded-core 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest1 patch 1 replyIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/1handle two-word commands with tmuxmeta/lib/oe/terminal.py+2 -20 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/1] tmux new does not take tw commands

This got routed to me because the visible error message says
Execution of "pseudo /bin/bash" failed

but actually it's nothing to do with pseudo. The real problem is that the tmux class in terminal.py is producing a string like:
'tmux new <args> {command}'

This then gets .format called on it, substituting in a command. If the comand is "pseudo /bin/bash", this is:
'tmux new <args> pseudo /bin/bash'
which is a usage error. It would be okay if it were:
'tmux new <args> "pseudo /bin/bash"'

So add double quotes. Of course, that assumes that <command> never has double quotes in it... If it does, that may need a fancier solution.

The following changes since commit b4f208f418d18f2a4e78a56bebacef481061d917:
Saul Wold (1):
tar: don't mv tar for nativesdk

are available in the git repository at:

git://git.yoctoproject.org/poky-contrib seebs/tmuxfix
http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=seebs/tmuxfix

Peter Seebach (1):
handle two-word commands with tmux

 meta/lib/oe/terminal.py |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)