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 617600 - Don't assume we are initialized if python is
Don't assume we are initialized if python is
Status: RESOLVED FIXED
Product: gedit
Classification: Applications
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Gedit maintainers
Gedit maintainers
Depends on:
Blocks: 620060
 
 
Reported: 2010-05-04 03:25 UTC by Garrett Regier
Modified: 2010-05-30 00:31 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Don't assume we are initialized if python is (3.58 KB, patch)
2010-05-04 03:25 UTC, Garrett Regier 		none Details | Review
Updated as comments suggested (3.79 KB, patch)
2010-05-04 07:07 UTC, Garrett Regier 		none Details | Review

Description Garrett Regier 2010-05-04 03:25:40 UTC 
Created attachment 160237 [details] [review]
Don't assume we are initialized if python is

If python is initialized the python loaded assumes that it is.

This causes issues when using gtkparasite which has its own python console that is initialized before gedit's.
Comment 1 jessevdk@gmail.com 2010-05-04 07:00:48 UTC 
Review of attachment 160237 [details] [review]:

Patch looks good, see minor comment below

::: plugin-loaders/python/gedit-plugin-loader-python.c
@@ +46,3 @@
+	STATUS_FAILED,
+	STATUS_SUCCESS
+} StatusType;

Could you make this into GeditPythonStatusType and GEDIT_PYTHON_STATUS_SUCCESS etc.
Comment 2 Garrett Regier 2010-05-04 07:07:31 UTC 
Created attachment 160246 [details] [review]
Updated as comments suggested
Comment 3 Garrett Regier 2010-05-05 04:07:08 UTC 
This problem has been fixed in the development version. The fix will be
available in the next major software release.