[patch] #40508 - keep ssh subprocess in same process group

Martin Pool mbp at sourcefrog.net
Wed Apr 26 06:05:05 BST 2006


On 26/04/2006, at 2:59 PM, Robert Collins wrote:

> On Wed, 2006-04-26 at 14:56 +1000, Martin Pool wrote:
>> This patch fixes #40508, and has an alternative fix for #5987.   
>> See the
>> diff for details/justification.  I will put this in 0.8 if it's
>> approved.
>
> this seems incomplete :
> we want to free locks on remote servers when we recieve:
> SIGHUP
> SIGINT
>
> A quick look through kill -l doesn't suggest any others - but its
> definately not just INT to ignore.

OK, I'll add SIGHUP as well.  Does this have your approval with that  
done?

Note that bzr itself does not handle SIGHUP at the moment and will  
need changes in a broader scope.    There is builtin Python support  
to catch SIGINT and raise an exception, but nothing similar for  
SIGHUP.  Anyhow I've filed

https://launchpad.net/products/bzr/+bug/40508

-- 
Martin







More information about the bazaar mailing list