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 389197 - Kill bonobo?
Kill bonobo?
Status: RESOLVED FIXED
Product: gthumb
Classification: Other
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: Paolo Bacchilega
Paolo Bacchilega
Depends on:
Blocks:
 
 
Reported: 2006-12-24 13:37 UTC by Björn Lindqvist
Modified: 2007-01-22 14:20 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Remove bonobo stuff from gThumb (14.83 KB, patch)
2006-12-31 02:57 UTC, Björn Lindqvist
none Details | Review

Description Björn Lindqvist 2006-12-24 13:37:48 UTC
gthumb uses some kind of bonobo magic to expose itself as a bonobo component. But, AFAIK, bonobo is going to be deprecated and nobody likes bonobo anymore. Plus, there are no apps that actually use gthumbs bonobo component for anything.

So I wonder, would it be a good idea to try and get rid of all bonobo parts from gthumb?
Comment 1 Michael Chudobiak 2006-12-24 20:45:24 UTC
It has already been removed from CVS HEAD, as of 2006-11-30.

- Mike
Comment 2 Björn Lindqvist 2006-12-30 15:49:38 UTC
It still has many references to bonobo. See for example http://cvs.gnome.org/viewcvs/gthumb/src/main.c?rev=1.56&view=markup.
Comment 3 Michael Chudobiak 2006-12-30 21:45:23 UTC
Yes, you are right. I should have said that the process of removing bonobo has started. Clean-up patches are welcome :-)

- Mike
Comment 4 Björn Lindqvist 2006-12-31 02:56:22 UTC
Here is an attempt to remove as much bonobo (and IDL and Orbit) as possible. I also deleted some bonobo-specific files and cvs diff can't handle changes like that.

 * data/GNOME_GThumb.server.in.in
 * src/GNOME_GThumb.idl
 * src/gth-application.c
 * src/gth-application.h

OT, but am I the only one finding it mildly ironic that the almost extinct ape is being exterminated in the GNOME project? :)
Comment 5 Björn Lindqvist 2006-12-31 02:57:15 UTC
Created attachment 79082 [details] [review]
Remove bonobo stuff from gThumb

Apply with "patch < name-of-it.patch -p0" in the root directory.
Comment 6 Michael Chudobiak 2007-01-01 17:40:23 UTC
Patch committed to svn (hopefully I did it right - we just switched from cvs).

Thank you for the patch!

- Mike
Comment 7 Paolo Bacchilega 2007-01-01 18:23:38 UTC
Please revert this change, bonobo is still used to provide a single instance application. 
Comment 8 Michael Chudobiak 2007-01-01 20:18:49 UTC
Patch reverted as requested.

- Mike
Comment 9 Paolo Bacchilega 2007-01-01 23:45:30 UTC
The gtkunique module can be used to provide a single instance application.  Once the migration to gtkunique is completed it will be possible to remove the bonobo dependency.
Comment 10 Michael Chudobiak 2007-01-02 18:04:40 UTC
Sorry, I misunderstood the code!

- Mike
Comment 11 Paolo Bacchilega 2007-01-22 00:17:25 UTC
ok, the patch has been committed now, and, as anticipated above, gtkunique is now used to provide a single instance application.  In the near future I'll make the gtkunique  dependency (and the feature it provides) optional.
Comment 12 Michael Chudobiak 2007-01-22 14:08:03 UTC
Paolo,

What is happening with gtkunique 1.0 - how are users supposed to get it? I only see it in svn (no 1.0 .tar.gz release files?).

- Mike

Comment 13 Paolo Bacchilega 2007-01-22 14:20:56 UTC
I think the 1.0 version will be released soon, it's also very likely that gtkunique will be included in gtk+ itself in the next development cycle of gtk+.