$OpenBSD: patch-shell_rb-python-module_c,v 1.1 2010/09/25 18:13:13 ajacoutot Exp $

From 4394826f36fad0ad36ea773b6d4525dfcfcd389b Mon Sep 17 00:00:00 2001
From: Jonathan Matthew <jonathan@d14n.org>
Date: Wed, 05 May 2010 12:58:26 +0000
Subject: python: fix a number of python initialization problems (bug #617587)

- pygtk.require("2.8") doesn't work - it's only after a major version,
  so we should pass in "2.0" instead
- init_pygobject() is deprecated, use pygobject_init (and pass in the
  version we require) instead
- init_pygtk() is a macro that returns from the current function on
  error, so we need to call it from a separate function for our error
  handling to work
- if some aspect of python initialization failed, we were still using
  the pygobject GIL macros, which were crashing

--- shell/rb-python-module.c.orig	Thu Mar 25 01:10:15 2010
+++ shell/rb-python-module.c	Sat Sep 25 19:55:32 2010
@@ -113,8 +113,16 @@ extern PyMethodDef pyrb_functions[];
 /* We retreive this to check for correct class hierarchy */
 static PyTypeObject *PyRBPlugin_Type;
 
+static gboolean python_init_successful;
+
 G_DEFINE_TYPE (RBPythonModule, rb_python_module, G_TYPE_TYPE_MODULE);
 
+static void
+actually_init_pygtk (void)
+{
+	init_pygtk ();
+}
+
 void
 rb_python_module_init_python (void)
 {
@@ -127,6 +135,7 @@ rb_python_module_init_python (void)
 	char *argv[] = { "rb", "rhythmdb", NULL };
 	GList *paths;
 
+	python_init_successful = FALSE;
 	if (Py_IsInitialized ()) {
 		g_warning ("Python Should only be initialized once, since it's in class_init");
 		g_return_if_reached ();
@@ -159,7 +168,7 @@ rb_python_module_init_python (void)
 
 	PySys_SetArgv (1, argv);
 
-	/* pygtk.require("2.8") */
+	/* pygtk.require("2.0") */
 	pygtk = PyImport_ImportModule ("pygtk");
 	if (pygtk == NULL) {
 		g_warning ("Could not import pygtk");
@@ -169,11 +178,15 @@ rb_python_module_init_python (void)
 
 	mdict = PyModule_GetDict (pygtk);
 	require = PyDict_GetItemString (mdict, "require");
-	PyObject_CallObject (require, Py_BuildValue ("(S)", PyString_FromString ("2.8")));
+	PyObject_CallObject (require, Py_BuildValue ("(S)", PyString_FromString ("2.0")));
+	if (PyErr_Occurred ()) {
+		g_warning ("pygtk.require(2.0) failed");
+		PyErr_Print();
+		return;
+	}
 
 	/* import gobject */
-	init_pygobject ();
-	if (PyErr_Occurred ()) {
+	if (pygobject_init (2, 16, 0) == NULL) {
 		g_warning ("Could not initialize pygobject");
 		PyErr_Print();
 		return;
@@ -188,7 +201,7 @@ rb_python_module_init_python (void)
 #endif
 
 	/* import gtk */
-	init_pygtk ();
+	actually_init_pygtk ();
 	if (PyErr_Occurred ()) {
 		g_warning ("Could not initialize pygtk");
 		PyErr_Print();
@@ -206,7 +219,7 @@ rb_python_module_init_python (void)
 
 	mdict = PyModule_GetDict (gtk);
 	pygtk_version = PyDict_GetItemString (mdict, "pygtk_version");
-	pygtk_required_version = Py_BuildValue ("(iii)", 2, 4, 0);
+	pygtk_required_version = Py_BuildValue ("(iii)", 2, 8, 0);
 	if (PyObject_Compare (pygtk_version, pygtk_required_version) == -1) {
 		g_warning("PyGTK %s required, but %s found.",
 				  PyString_AsString (PyObject_Repr (pygtk_required_version)),
@@ -298,6 +311,8 @@ rb_python_module_init_python (void)
 	gettext_args = Py_BuildValue ("ss", GETTEXT_PACKAGE, GNOMELOCALEDIR);
 	PyObject_CallObject (install, gettext_args);
 	Py_DECREF (gettext_args);
+
+	python_init_successful = TRUE;
 }
 
 static gboolean
@@ -363,6 +378,11 @@ rb_python_module_load_with_gil (GTypeModule *module)
 	PyGILState_STATE state;
 	gboolean ret;
 
+	if (python_init_successful == FALSE) {
+		g_warning ("unable to load module as python runtime could not be initialized");
+		return FALSE;
+	}
+
 	state = pyg_gil_state_ensure ();
 	ret = rb_python_module_load (module);
 	pyg_gil_state_release (state);
@@ -517,6 +537,12 @@ rb_python_module_new (const gchar *path,
 	g_type_module_set_name (G_TYPE_MODULE (result), module);
 
 	return result;
+}
+
+gboolean
+rb_python_init_successful (void)
+{
+	return python_init_successful;
 }
 
 /* --- these are not module methods, they are here out of convenience --- */
