Re: [PATCH] Add wait-for signaling to commands that create new panes

2014-08-24 Thread Nathaniel W Filardo
On Mon, Aug 11, 2014 at 11:19:19PM +0100, Nicholas Marriott wrote: > Hi > > Can't you just run "mycommand; tmux wait -S foo" in the pane? That's not robust against the shell's process group or tmux itself being fatally signaled, which would kill both the "mycommand" process and prevent the shell

Re: [PATCH] Add wait-for signaling to commands that create new panes

2014-08-11 Thread Nicholas Marriott
You could use trap in the shell to protect against the first case, and I'm not convinced the new flag is useful enough beyond this very specific case. The second is a known issue which could and should be fixed as a separate change regardless of whether not we add any new flags. There have been p

Re: [PATCH] Add wait-for signaling to commands that create new panes

2014-08-11 Thread Nicholas Marriott
Hi Can't you just run "mycommand; tmux wait -S foo" in the pane? On Thu, Aug 07, 2014 at 04:15:15PM -0400, Nathaniel W Filardo wrote: > On Thu, Aug 07, 2014 at 09:05:01PM +0100, Nicholas Marriott wrote: > > Hi > > > > What is this useful for? > > My use case is that I want to run half a dozen

Re: [PATCH] Add wait-for signaling to commands that create new panes

2014-08-08 Thread Nathaniel W Filardo
On Mon, Aug 04, 2014 at 10:59:21PM +0100, Thomas Adam wrote: >[useful feedback] Here's take 2; it should (unless I goofed) incorporate all your feedback. Thanks! --nwf; diff --git a/cmd-new-session.c b/cmd-new-session.c index b36de70..d23fea1 100644 --- a/cmd-new-session.c +++ b/cmd-new-session.c

Re: [PATCH] Add wait-for signaling to commands that create new panes

2014-08-08 Thread Nathaniel W Filardo
On Thu, Aug 07, 2014 at 09:05:01PM +0100, Nicholas Marriott wrote: > Hi > > What is this useful for? My use case is that I want to run half a dozen or so jobs via runit and I wanted them to share a tmux session for easy administrative access rather than each spawning their own. But I need some w

Re: [PATCH] Add wait-for signaling to commands that create new panes

2014-08-07 Thread Nicholas Marriott
Hi What is this useful for? This seems like a job for hooks if they are ever finished. On Mon, Aug 04, 2014 at 01:59:26PM -0400, Nathaniel W Filardo wrote: > Greetings list, > > The attached patch adds options to new-window, new-session, and split-window > (commands which create panes) to sign

Re: [PATCH] Add wait-for signaling to commands that create new panes

2014-08-04 Thread Thomas Adam
On Mon, Aug 04, 2014 at 01:59:26PM -0400, Nathaniel W Filardo wrote: > Greetings list, > > The attached patch adds options to new-window, new-session, and split-window > (commands which create panes) to signal a wait_channel when that pane is > destroyed, for example after the contained process ex