[Expired for gnome-session (Ubuntu) because there has been no activity
for 60 days.]

** Changed in: gnome-session (Ubuntu)
       Status: Incomplete => Expired

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to gnome-session in Ubuntu.
https://bugs.launchpad.net/bugs/1101154

Title:
  all signal handlers have been removed in error

Status in “gnome-session” package in Ubuntu:
  Expired

Bug description:
  gnome-session/main.c has disabled all signal handling, but intention
  was seemingly to only disable SIGSEGV signal handing:

  10:43 <jodh> chrisccoulson: hi - I'm looking at how gnome-session currently
               shuts down and trying to understand why the following is
               commented out: http://paste.ubuntu.com/1544640/
  10:44 <chrisccoulson> jodh, IIRC it was to stop gnome-session from swallowing
                        those signals so we got crash reports with apport
  10:44 <chrisccoulson> but this was a long time ago ;)
  10:49 <jodh> chrisccoulson: ok, thanks.
  10:49 <chrisccoulson> jodh, technically, i think only the call to
                        gdm_signal_handler_add_fatal() needs to be commented out
  10:49 <chrisccoulson> i can't remember why we disabled all signal handling
  10:50 <chrisccoulson> i've slept and drank beer since then ;)
  10:50 <pitti> chrisccoulson: or perhaps a merge error? in the original patch
                the commented out section might only have been for SIGSEGV, and
                in newer upstream versions the SIGTERMs got added?
  10:52 <chrisccoulson> pitti, yeah, you're right
  10:52 <chrisccoulson> the original patch was
                        
http://bazaar.launchpad.net/~ubuntu-desktop/gnome-session/ubuntu/revision/18
  10:52 <chrisccoulson> jodh ^^
  10:53 <chrisccoulson> it got changed here by seb128:
                        
http://bazaar.launchpad.net/~ubuntu-desktop/gnome-session/ubuntu/revision/233
  10:53 <chrisccoulson> not sure why ;)
  10:53 <pitti> that doesn't look intended
  10:54 <chrisccoulson> jodh, so, the short summary is - the patch is only meant
                        to disable the SEGV handler ;)

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/gnome-session/+bug/1101154/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to