After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 622442 - Make GObject criticals fatal
Make GObject criticals fatal
Status: RESOLVED WONTFIX
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2010-06-22 20:59 UTC by Colin Walters
Modified: 2013-05-31 16:56 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Make GObject criticals fatal (1.16 KB, patch)
2010-06-22 20:59 UTC, Colin Walters
rejected Details | Review

Description Colin Walters 2010-06-22 20:59:33 UTC
This turns them into crashes, so ABRT will pick them up, we'll
get more stack traces, and we'll fix them.
Comment 1 Colin Walters 2010-06-22 20:59:35 UTC
Created attachment 164349 [details] [review]
Make GObject criticals fatal
Comment 2 Owen Taylor 2010-06-24 17:35:40 UTC
Review of attachment 164349 [details] [review]:

::: src/gnome-shell-plugin.c
@@ +315,3 @@
+   * Assume the OS is shipping with a crash catching system.
+   */
+  g_log_set_fatal_mask ("GLib-GObject", G_LOG_LEVEL_CRITICAL);

Why this one domain?
Comment 3 William Jon McCann 2010-06-24 18:43:48 UTC
To add some context I was hoping to be able to catch errors like:
Window manager warning: Log level 16: invalid uninstantiatable type `(null)' in cast to `GObject'
Window manager warning: Log level 8: g_object_run_dispose: assertion `G_IS_OBJECT (object)' failed
Window manager warning: Log level 8: g_object_unref: assertion `G_IS_OBJECT (object)' failed
Window manager warning: Log level 16: invalid uninstantiatable type `(null)' in cast to `GObject'
Window manager warning: Log level 8: g_object_run_dispose: assertion `G_IS_OBJECT (object)' failed
Window manager warning: Log level 8: g_object_unref: assertion `G_IS_OBJECT (object)' failed
Comment 4 Jasper St. Pierre (not reading bugmail) 2011-08-03 18:28:54 UTC
(In reply to comment #3)
> To add some context I was hoping to be able to catch errors like:
> Window manager warning: Log level 16: invalid uninstantiatable type `(null)' in
> cast to `GObject'
> Window manager warning: Log level 8: g_object_run_dispose: assertion
> `G_IS_OBJECT (object)' failed
> Window manager warning: Log level 8: g_object_unref: assertion `G_IS_OBJECT
> (object)' failed
> Window manager warning: Log level 16: invalid uninstantiatable type `(null)' in
> cast to `GObject'
> Window manager warning: Log level 8: g_object_run_dispose: assertion
> `G_IS_OBJECT (object)' failed
> Window manager warning: Log level 8: g_object_unref: assertion `G_IS_OBJECT
> (object)' failed

G_DEBUG=fatal_criticals ?
Comment 5 Colin Walters 2012-05-03 18:17:01 UTC
Comment on attachment 164349 [details] [review]
Make GObject criticals fatal

Um...so I guess let's defer this pending some more GNOME-global decision on making GObject criticals fatal rather than handling them per-component.
Comment 6 Alexandre Franke 2013-05-31 10:00:35 UTC
(In reply to comment #5)
> (From update of attachment 164349 [details] [review])
> Um...so I guess let's defer this pending some more GNOME-global decision on
> making GObject criticals fatal rather than handling them per-component.

Is that topic discussed somewhere we can track?
Comment 7 Colin Walters 2013-05-31 14:15:26 UTC
(In reply to comment #6)
> (In reply to comment #5)
> > (From update of attachment 164349 [details] [review] [details])
> > Um...so I guess let's defer this pending some more GNOME-global decision on
> > making GObject criticals fatal rather than handling them per-component.
> 
> Is that topic discussed somewhere we can track?

I am not aware of anyone discussing this at the moment.  Were we to try, there's lots of criticals to fix first that happen just on bootup in a standard qemu configuration...

(How/why did you find this bug anyways?)
Comment 8 Alexandre Franke 2013-05-31 14:36:31 UTC
(In reply to comment #7)
> (How/why did you find this bug anyways?)

I was just trying to do some triaging. :)
Comment 9 Florian Müllner 2013-05-31 14:48:51 UTC
(In reply to comment #8)
> I was just trying to do some triaging. :)

... which is very much appreciated!
Comment 10 Jasper St. Pierre (not reading bugmail) 2013-05-31 16:56:34 UTC
I think G_DEBUG=fatal-criticals is a good enough solution, we don't really need a patch for this.