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] Implement command to enable / disable input

2014-08-11 Thread Anish Athalye
Is there anything else that needs to be done to this patch? Is it likely to be integrated any time soon? Thanks, Anish > On Aug 7, 2014, at 1:52 PM, Anish Athalye wrote: > > Thanks for the quick feedback! I've attached an updated patch. > > –Anish > > <0001-Add-feature-to-enable-or-disable-k

Re: [PATCH] copy-mode: prevent switching to other end multiple times

2014-08-11 Thread Nicholas Marriott
Applied thanks (although your mailer mangled the patch). On Mon, Jun 23, 2014 at 06:31:45PM +0300, Juho Pohjala wrote: > Hi > > Assuming vi-keys and copy mode with some selected text. Switching to > other end 333 times (i.e. typing '333o' takes a while). As unlikely as > it is that a user uses r

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] Fix off by one error

2014-08-11 Thread Nicholas Marriott
Applied thanks On Sun, Aug 10, 2014 at 09:49:14AM +0100, Balazs Kezes wrote: > On 2014-06-07 15:16 +0100, Balazs Kezes wrote: > > OK, I've attached a patch which fixes the behavior for the vi mode and > > also fixes the left key bug for the emacs mode. Please test both > > modes! > > Hi Nicholas

Re: [PATCH] Implement command to enable / disable input

2014-08-11 Thread Nicholas Marriott
Applied to OpenBSD with some minor tweaks, thanks On Thu, Aug 07, 2014 at 01:52:21PM -0700, Anish Athalye wrote: > Thanks for the quick feedback! I've attached an updated patch. > > ???Anish > > > > > On Aug 7, 2014, at 1:01 PM, Nicholas Marriott > > wrote: > > > > Ok but in that case yo

Re: flock again

2014-08-11 Thread Nicholas Marriott
I'd add a configure.ac test though and use it on any platform w/o flock not just Solaris. Original message From: Nicholas Marriott Date: 11/08/2014 09:17 (GMT+00:00) To: Dagobert Michelsen Cc: tmux-users@lists.sourceforge.net Subject: Re: flock again Yes it will have

Re: flock again

2014-08-11 Thread Nicholas Marriott
Yes it will have to be a stub, fcntl is useless too. Original message From: Dagobert Michelsen Date: 11/08/2014 08:56 (GMT+00:00) To: Nicholas Marriott Cc: tmux-users@lists.sourceforge.net Subject: Re: flock again Hi Nicholas, Am 10.08.2014 um 10:54 schrieb Nicholas M

Re: flock again

2014-08-11 Thread Dagobert Michelsen
Hi Nicholas, Am 10.08.2014 um 10:54 schrieb Nicholas Marriott : > lockf and fcntl are broken (any close on the fd releases all locks) so I > suggest the best option is to create an flock in compat/ that is a no-op and > do without the locking on Solaris. > Or there may be a locking function that