send-keys not working with a target that has a . in the session name.

2013-03-06 Thread Wraithan (Chris McDonald)
(pytmux)wraithbook:pytmux cmcdonald$ tmux send-keys -t wraithan.net:3 "workon wraithan.net && git pull" ^M session not found: wraithan (pytmux)wraithbook:pytmux cmcdonald$ tmux send-keys -t 'wraithan.net':3 "workon wraithan.net && git pull" ^M session not found: wraithan (pytmux)wraithbook:pytmux

Hidden pane sometimes 'leaking' output into zoomed window

2013-03-06 Thread Thiago Padilha
I could not find the exact conditions that cause this, but here certain facts about my workflow 1- split window with top pane running shell or node.js server web server 2- bottom pane running vim, which is most of the time zoomed 3- when I edit and save a file in vim, a background watcher process

Re: [PATCH] Pass valid pointer to grid_string_cells or it segfaults

2013-03-06 Thread eugene everson
Understand, thanks for the explanation. On Wed, Mar 06, 2013 at 11:26:30AM +, Nicholas Marriott wrote: > On Wed, Mar 06, 2013 at 08:02:49PM +0900, eugene everson wrote: > > Hi Nicholas, > > > > First just quick want to say thanks first, switched yesterday to tmux and > > it's been so sweet. A

Re: [PATCH] Pass valid pointer to grid_string_cells or it segfaults

2013-03-06 Thread Nicholas Marriott
On Wed, Mar 06, 2013 at 08:02:49PM +0900, eugene everson wrote: > Hi Nicholas, > > First just quick want to say thanks first, switched yesterday to tmux and > it's been so sweet. And also love the source code. Thanks. > > About the patch, this is a bit nitpicking so if you're busy think you can >

Re: [PATCH] Pass valid pointer to grid_string_cells or it segfaults

2013-03-06 Thread eugene everson
Hi Nicholas, First just quick want to say thanks first, switched yesterday to tmux and it's been so sweet. And also love the source code. Thanks. About the patch, this is a bit nitpicking so if you're busy think you can leave this safely for later. - Later in grid_string_cells if(with_codes) is

Re: [PATCH] Implement 'wait-for' command

2013-03-06 Thread Nicholas Marriott
Looks good, applied, thanks. On Wed, Mar 06, 2013 at 07:49:14AM -0300, Thiago Padilha wrote: > Here it is. My first patch contains a example shell script in case you > want to add. > --- > cmd-wait-for.c | 81 > ++ > tmux.1 | 12 ++

Re: [PATCH] Implement 'wait-for' command

2013-03-06 Thread Thiago Padilha
Here it is. My first patch contains a example shell script in case you want to add. --- cmd-wait-for.c | 81 ++ tmux.1 | 12 ++--- 2 files changed, 85 insertions(+), 8 deletions(-) diff --git a/cmd-wait-for.c b/cmd-wait-for.c inde

Re: [PATCH] Implement 'wait-for' command

2013-03-06 Thread Nicholas Marriott
No I don't think it is needed. On Wed, Mar 06, 2013 at 07:31:29AM -0300, Thiago Padilha wrote: > Do you want me to add the example script? > > On Wed, Mar 6, 2013 at 7:09 AM, Thiago Padilha wrote: > > Yes will do in a minute > > > > On Wed, Mar 6, 2013 at 6:58 AM, Nicholas Marriott > > wrote:

Re: [PATCH] Implement 'wait-for' command

2013-03-06 Thread Thiago Padilha
Do you want me to add the example script? On Wed, Mar 6, 2013 at 7:09 AM, Thiago Padilha wrote: > Yes will do in a minute > > On Wed, Mar 6, 2013 at 6:58 AM, Nicholas Marriott > wrote: >> Committed, thanks. Do you want to tweak the lock bits to fit too? >> >> >> On Wed, Mar 06, 2013 at 06:53:12A

Re: [PATCH] Implement 'wait-for' command

2013-03-06 Thread Thiago Padilha
Yes will do in a minute On Wed, Mar 6, 2013 at 6:58 AM, Nicholas Marriott wrote: > Committed, thanks. Do you want to tweak the lock bits to fit too? > > > On Wed, Mar 06, 2013 at 06:53:12AM -0300, Thiago Padilha wrote: >> This looks good to me >> >> On Wed, Mar 6, 2013 at 6:37 AM, Nicholas Marrio

Re: [PATCH] Implement 'wait-for' command

2013-03-06 Thread Nicholas Marriott
Committed, thanks. Do you want to tweak the lock bits to fit too? On Wed, Mar 06, 2013 at 06:53:12AM -0300, Thiago Padilha wrote: > This looks good to me > > On Wed, Mar 6, 2013 at 6:37 AM, Nicholas Marriott > wrote: > > Ok this looks pretty good. I've made a few changes but mainly style/layout

Re: [PATCH] Pass valid pointer to grid_string_cells or it segfaults

2013-03-06 Thread Nicholas Marriott
Thanks - fixed by checking in grid_string_cells instead. On Wed, Mar 06, 2013 at 10:24:55AM +0900, eugene everson wrote: > Reproducible with :find-window .. (f) > --- > grid-view.c |4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/grid-view.c b/grid-view.c > index

Re: [PATCH] Implement 'wait-for' command

2013-03-06 Thread Thiago Padilha
This looks good to me On Wed, Mar 6, 2013 at 6:37 AM, Nicholas Marriott wrote: > Ok this looks pretty good. I've made a few changes but mainly style/layout > nits: > > - I don't think we need find and find_and_create functions, the find > will only be done once and the create twice (when there

Re: [PATCH] Implement 'wait-for' command

2013-03-06 Thread Nicholas Marriott
Ok this looks pretty good. I've made a few changes but mainly style/layout nits: - I don't think we need find and find_and_create functions, the find will only be done once and the create twice (when there is lock too). - A single cmdq can only be waiting once so no need to have a wrapper str