Tyler2004 / pychess

Automatically exported from code.google.com/p/pychess
GNU General Public License v3.0
1 stars 0 forks source link

Crash on startup because of missing gtk.gdk.threads_enter() #919

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Start pychess as packaged on Mageia (pristine 0.12rc2 with a patch for the 
location of gtksourceview2)
2. It crashes with the following output:

$ pychess 
** Message: pygobject_register_sinkfunc is deprecated (GstObject)

(pychess:16371): GLib-GObject-CRITICAL **: g_object_set_qdata: assertion 
'G_IS_OBJECT (object)' failed
Attempt to unlock mutex that was not locked
Abandon

A fellow Mageia contributor is writing a patch to fix this, I'll post it here 
as soon as it's ready. It seems to be missing a call to missing 
gtk.gdk.threads_enter() (and missing gtk.gdk.threads_leave()), though those 
seem to be deprecated: 
https://developer.gnome.org/gdk3/stable/gdk3-Threads.html#gdk-threads-init

Did you use PyChess from a repo, tarball, install or hg?
Packaged version of PyChess 0.12rc2 for Mageia Cauldron. The spec file can be 
seen here: 
http://svnweb.mageia.org/packages/cauldron/pychess/current/SPECS/pychess.spec?vi
ew=markup&pathrev=728000

Please attach the latest pychess logfile from ~/.local/share/pychess/
http://paste.fedoraproject.org/137133/11908143

Original issue reported on code.google.com by rverschelde on 28 Sep 2014 at 12:47

GoogleCodeExporter commented 9 years ago
Does the attached patch work?

Original comment by mattgatto on 28 Sep 2014 at 12:59

Attachments:

GoogleCodeExporter commented 9 years ago
Here is the patch proposed by wally from the Mageia packagers team: 
http://svnweb.mageia.org/packages/cauldron/pychess/current/SOURCES/pychess-0.12r
c2-gdk.threads_init.patch?view=markup

Your patch is working too, so I'll let your see if yours is sufficient or 
should be merged with wally's.

Original comment by rverschelde on 28 Sep 2014 at 1:21

GoogleCodeExporter commented 9 years ago
Were aware about threads_enter being deprecated. That is being handled in issue 
914.

Original comment by mattgatto on 28 Sep 2014 at 1:23

GoogleCodeExporter commented 9 years ago
This issue was closed by revision 937fe0c38d65.

Original comment by mattgatto on 6 Nov 2014 at 7:51