On Wed, 2015-01-14 at 14:48 +0100, Jakub Filak wrote:
> On Wednesday 14 of January 2015 14:19:44 Bastien Nocera wrote:
> > On Wed, 2015-01-14 at 14:09 +0100, Jakub Filak wrote:
> > > On Wednesday 14 of January 2015 12:06:21 Bastien Nocera wrote:
> > > > None of the actions we're supposed to be taking after the main loop
> > > > should be necessary at that point in time. Let GTK+ handle that as it
> > > > should by exiting if the SIGTERM signal is sent.
> > > 
> > > Can you please point me to some documentation dealing with the signals and
> > > GTK+?
> > 
> > There's no documentation, there's source code though. It doesn't handle
> > SIGTERM especially, so would just "terminate" if sent SIGTERM.
> > 
> > > I applied your patch, started abrt-applet, sent SIGTERM to it and the
> > > process was "Terminated" but it should gracefully exit.
> > > 
> > > I use SIGTERM to test the applet's behaviour when Desktop is terminating.
> > 
> > What would abrt-applet do on exit that's required to be done on exit?
> 
> abrt-applet notifies a user about problems that occurred while his system was 
> not running a desktop session. The applet has a list where it stores the seen 
> problem IDs and the applet sends a notification for all not-yet-seen problems 
> at start-up. The seen list must be updated before the applet exits because of 
> unclear legacy reasons I am not able to recollect and are possibly gone, but 
> I'm not sure and I'd rather leave it as it is.

Keeping a list of seen items seems like the sort of thing that the
application crashing/going away suddenly shouldn't affect. Why don't we
remove it now, and see whether it causes any problems instead?

Reply via email to