[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: [PATCH] Reset signal handlers in KProcess child
From:       Dmitry Suzdalev <dimsuzkde () gmail ! com>
Date:       2008-01-26 8:52:10
Message-ID: 200801261451.18144.dimsuz () gmail ! com
[Download RAW message or body]

Hi Robert!

On Saturday 26 January 2008 07:22:55 Robert Knight wrote:
> Hi,
>
> K3Process used to reset signal handler for all signals to SIG_DFL just
> after forking.  QProcess/KProcess currently do not do this.  This
> caused a problem in Konsole where pressing Ctrl+C did terminate many
> programs run in the shell because they inherited a SIG_IGN handler for
> the SIGINT signal from the shell which inherited it from Konsole.
> Konsole ultimately inherited SIGINT from somewhere else, it looks like
> KDM might be the cause (See process.c:Fork)
I can confirm that your patch fixes Ctrl-C for me.
Yay! :)

If (as Thiago says) it can't make it to kprocess, perhaps it can be placed in 
qt-copy temporarily?

Anyway, for now I'll keep it as a local change and be happy :)

Thanks for fixing this!

Cheers,
Dmitry.
[prev in list] [next in list] [prev in thread] [next in thread]