GNOME Bugzilla – Bug 310370
crash sometimes on session startup
Last modified: 2010-01-24 01:07:04 UTC
since 2.11.2 stickynotes crashes sometime on session startup: Backtrace was generated from '/usr/libexec/stickynotes_applet' Using host libthread_db library "/lib/tls/i686/cmov/libthread_db.so.1". [Thread debugging using libthread_db enabled] [New Thread -1222989600 (LWP 17013)] 0xffffe410 in ?? ()
+ Trace 61735
Thread 1 (Thread -1222989600 (LWP 17013))
According to simple-dup-finder this is a unique stack trace.
*** Bug 310896 has been marked as a duplicate of this bug. ***
there is an ubuntu bugzilla about that: http://bugzilla.ubuntu.com/show_bug.cgi?id=14512
This seems to be realted to NAUTILUS_DESKTOP_WINDOW_ID X atom not being set on time, or not being set at all if nautilus has been removed from the session. I am bitten by it because I routinely disable nautilus for my gnome session.
I'll take the liberty to add jaap@haitsma.org to the cc list because it seems this has to to do with the changes introduced in response to #308701
Sorry about the bug I have introduced. I'll try and find out how to solve it. I'm not completely sure if I can fix it. I'm not very experienced with Gtk and X. So any help is appreciated
Created attachment 53210 [details] [review] Patch possibly fixing crash I can't reproduce the crash on my machine, but I think this patch might fix the crash that people are seeing. Yanko can you try out the patch
Yes, the patch fixes it, thanks. The way to reporduce the crash is: # gnome-session-remove nautilus # gnome-session-save logout - loging (so that X forgets the atom, don't know if there are ways to do that from the commandline) stickynotes crashes on startup if its in the session or crashes when adding it to the panel if its not in the session.
Created attachment 53475 [details] [review] Patch fixing also desktop click This patch fixes the crash and also fixes the fact that the notes seemed to hide auto magically when you were logging in or if you use nautilus in browse mode and hit the back or forward button.
Ping can somebody review (and apply) this patch that fixes a crasher? Thanks
This looks visually clean. I haven't tested it yet. Earmarked for 2.12.2 and 2.13.
Ping. Would be nice if this could go in before 2.13.2 (release today) and 2.12.2(in 14 days)
Ok. This looks good. It no longer crashes when Nautilus isn't running, of course, there is no way to make the notes vanish when Nautilus isn't running. Stickynotes also do not remember their state when the applet is restarted (whether they were hidden or showing).
Ok. This is committed to HEAD. I will look at backporting it in a couple of weeks.
> Ok. This looks good. It no longer crashes when Nautilus isn't running, of > course, there is no way to make the notes vanish when Nautilus isn't running. Maybe add a hide notes menu item to the context menu??? > Stickynotes also do not remember their state when the applet is restarted > (whether they were hidden or showing) Always showing them on startup is actually a desired feature (at least for me). That way everytime I start up my PC I get reminded of things I should do. Furthermore it's very easy to move them out of your sight. By just launching an app or clicking on the desktop. Backporting should not really be a problem. If I remember correctly it applies cleanly on 2.12
> Always showing them on startup is actually a desired feature (at least for me). As a note, that's the reason which made me switch to tomboy on month ago, I have quite a lot of note and it's pretty annoying to have to click on the desktop every time on startup to mask them. It's also annoying if you hack on the panel and restart it quite often to try changes
OK I guess it depends on how you use it. I guess the best solution is to have sticky notes remember if it was visible or not and have a checkbox in the preferences for people like me who want them visible at startup.
Yeah. Restarting the panel made me comment on it. That is a simple enough feature that it could have a check box in the UI. [X] Always show sticky notes on startup It could be on by default. A hide option in the context menu is also a good idea. We should keep the minimise animation when you use the context menu hide also.
*** Bug 318526 has been marked as a duplicate of this bug. ***
*** Bug 318299 has been marked as a duplicate of this bug. ***
David as I'm seeing 2.12.2 tarballs coming out, just a reminder to apply this patch to 2.12.2
It should have been. Can you confirm for me please?
Confirmed it's there
*** Bug 322970 has been marked as a duplicate of this bug. ***
Marking as a possible target for gnome-love and retitling to capture the two new requirements of the bug: 1. Have an option in the menu to hide all of the stickynotes; 2. Have an option in the preferences to show all of the stickynotes on load
Found out that at least half of this is a duplicate of #310246. Closing this bug, since it is long and really, actually unrelated.
*** Bug 323860 has been marked as a duplicate of this bug. ***
*** Bug 324792 has been marked as a duplicate of this bug. ***
*** Bug 325106 has been marked as a duplicate of this bug. ***
*** Bug 325530 has been marked as a duplicate of this bug. ***
*** Bug 316544 has been marked as a duplicate of this bug. ***
*** Bug 320195 has been marked as a duplicate of this bug. ***
*** Bug 326619 has been marked as a duplicate of this bug. ***
*** Bug 327011 has been marked as a duplicate of this bug. ***
*** Bug 328482 has been marked as a duplicate of this bug. ***
*** Bug 330814 has been marked as a duplicate of this bug. ***
*** Bug 336615 has been marked as a duplicate of this bug. ***
*** Bug 342135 has been marked as a duplicate of this bug. ***
*** Bug 350832 has been marked as a duplicate of this bug. ***
*** Bug 363356 has been marked as a duplicate of this bug. ***