Just a little while ago I sent a version of this patch that fixes the shutdown option. I also found that the call to XCreateWindow in is_already_running was causing a problem with the Xserver because we were passing in the valuemask of CWEventMask but we weren't setting the attr.event_mask, which is not valid. I didn't catch this before because the problem was only showing up when you didn't compile with debug.
Therefore, I am now setting the mask to NULL to indicate that this window (which is simply setting an atom and doesn't need to respond to events - the atom is used to see if the program is already running or not). Anyway, here is the updated patch. I committed this patch to the GNOME 2.16 stable branch. Brian -------------- next part -------------- A non-text attachment was scrubbed... Name: gnome-sys-suspend-01-fixshutdown.diff Type: text/x-patch Size: 1089 bytes Desc: not available URL: <http://mail.opensolaris.org/pipermail/jds-review/attachments/20070413/da54bd83/attachment.bin>
