Patchwork TmuxRunning: handle multi-word commands

login
register
mail settings
Submitter Tyler Hall
Date June 30, 2013, 7:51 p.m.
Message ID <1372621913-13545-1-git-send-email-tyhall@lexmark.com>
Download mbox | patch
Permalink /patch/52699/
State Accepted
Commit 85186302deecc91a345e30598ea84521d8097b77
Headers show

Comments

Tyler Hall - June 30, 2013, 7:51 p.m.
From: Tyler Hall <tylerwhall@gmail.com>

Just as in f8ed7446755eeb88191e16749350efa1e7e6197c, tmux wants a single
argument for its command. This applies to the "split-window" command as
well as "new."

Note that this alone is not enough to fix the TmuxRunning devshell when
using pseudo because tmux does not preserve the environment that pseudo
requires.

Signed-off-by: Tyler Hall <tylerwhall@gmail.com>
---
 meta/lib/oe/terminal.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Patch

diff --git a/meta/lib/oe/terminal.py b/meta/lib/oe/terminal.py
index 25f8004..4502f34 100644
--- a/meta/lib/oe/terminal.py
+++ b/meta/lib/oe/terminal.py
@@ -108,7 +108,7 @@  class Screen(Terminal):
 class TmuxRunning(Terminal):
     """Open a new pane in the current running tmux window"""
     name = 'tmux-running'
-    command = 'tmux split-window {command}'
+    command = 'tmux split-window "{command}"'
     priority = 2.75
 
     def __init__(self, sh_cmd, title=None, env=None, d=None):