Re: Implementing some extra features

2013-02-23 Thread Thiago Padilha
Ok thanks for the info. I just got home now and will go sleep, and when I wake up you will probably be gone already, so I will see what I can do and next sunday you give feedback. Until then On Sat, Feb 23, 2013 at 7:45 PM, Nicholas Marriott < nicholas.marri...@gmail.com> wrote: > I've committe

Re: Implementing some extra features

2013-02-23 Thread Nicholas Marriott
I've committed my cmdq changes after all so you will need to do it that way at some point, the differences aren't substantial - basically what I outlined below, save and reference the cmdq rather than the client and wake it up with cmdq_free/cmdq_continue. If you look at run-shell it should be clea

Re: Implementing some extra features

2013-02-23 Thread Nicholas Marriott
On Sat, Feb 23, 2013 at 06:24:56PM -0300, Thiago Padilha wrote: >I have started to read the source code and began to wonder if those >features are possible to implement. >I'm assuming that a overview(very high level) of what happens when a tmux >command is executed is: Yes this is

Re: Implementing some extra features

2013-02-23 Thread Thiago Padilha
I have started to read the source code and began to wonder if those features are possible to implement. I'm assuming that a overview(very high level) of what happens when a tmux command is executed is: 1- user types 'tmux somecmd' in a shell running inside a tmux session, which opens 'tmux progra

Re: [PATCH] Expand variables in the run-shell command string

2013-02-23 Thread Nicholas Marriott
Applied with some minor tweaks, and changed to add -t to if-shell too. Thanks. On Sat, Feb 23, 2013 at 03:29:55PM -0300, Thiago Padilha wrote: > Ok here is the updated patch > > --- > cmd-if-shell.c | 20 ++-- > cmd-run-shell.c | 18 ++ > tmux.1 | 10 ++

Re: [PATCH] Expand variables in the run-shell command string

2013-02-23 Thread Thiago Padilha
Ok here is the updated patch --- cmd-if-shell.c | 20 ++-- cmd-run-shell.c | 18 ++ tmux.1 | 10 -- 3 files changed, 40 insertions(+), 8 deletions(-) diff --git a/cmd-if-shell.c b/cmd-if-shell.c index 6f0b151..011974f 100644 --- a/cmd-if-shell.c +

Re: [PATCH] Expand variables in the run-shell command string

2013-02-23 Thread Nicholas Marriott
Hi Thanks but can you do the same for the if-shell shell-command as well? In the manpage you need to markup FORMATS as .Sx FORMATS (there are some examples already in the file). On Sat, Feb 23, 2013 at 01:45:43PM -0300, Thiago Padilha wrote: > Let's say I want to write a shell script that opera

[PATCH] Expand variables in the run-shell command string

2013-02-23 Thread Thiago Padilha
Let's say I want to write a shell script that operates on the currently focused pane, to be executed with 'run-shell'. The only way to do that for now is to call 'tmux list-panes' inside the script and parse the output to determine the active pane. This patches modifies run-shell to expand tmux #{v

Re: [PATCH 0/4] Add handling of client focus tracking

2013-02-23 Thread Nicholas Marriott
Applied now thanks. On Fri, Feb 22, 2013 at 11:59:33PM -0800, Aaron Jensen wrote: >A single patch is attached as well for easy application. > >On Friday, February 22, 2013 at 11:54 PM, Aaron Jensen wrote: > > Add handling of client focus tracking. > Also trying a different sen

Re: [PATCH 0/4] Add handling of client focus tracking

2013-02-23 Thread Nicholas Marriott
This looks good, the only problem is you need to skip where c == NULL || c->session == NULL in the loop server_client_check_focus in because the array can contain NULL clients but don't worry I will fix that. On Fri, Feb 22, 2013 at 11:59:33PM -0800, Aaron Jensen wrote: >A single patch is at

Re: [PATCH 0/4] Add handling of client focus tracking

2013-02-23 Thread Aaron Jensen
A single patch is attached as well for easy application. On Friday, February 22, 2013 at 11:54 PM, Aaron Jensen wrote: > Add handling of client focus tracking. > > Also trying a different send-email approach, let me know how it works. > > Aaron Jensen (4): > Turn bracketed paste mode on and o