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 622739 - Make bonobo optional
Make bonobo optional
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: general
2.31.x
Other All
: Normal normal
: 2.32.0
Assigned To: Joanmarie Diggs (IRC: joanie)
Orca Maintainers
Depends on:
Blocks:
 
 
Reported: 2010-06-25 15:44 UTC by Joanmarie Diggs (IRC: joanie)
Modified: 2010-09-20 10:55 UTC
See Also:
GNOME target: 3.0
GNOME version: ---


Attachments
first attempt (6.54 KB, patch)
2010-06-25 15:50 UTC, Joanmarie Diggs (IRC: joanie)
none Details | Review
take 2 (6.85 KB, patch)
2010-06-25 16:40 UTC, Joanmarie Diggs (IRC: joanie)
committed Details | Review

Description Joanmarie Diggs (IRC: joanie) 2010-06-25 15:44:10 UTC
We don't want to completely eliminate bonobo from Orca because we have -- and will continue to have -- users of older versions of GNOME running the latest version of Orca.* Still bonobo's out the door for GNOME 3.0. Therefore, it needs to be an optional dependency rather than a required one.

* While officially that's not supported, it typically works well. Therefore, we will continue to do our best to make that possible.
Comment 1 Joanmarie Diggs (IRC: joanie) 2010-06-25 15:50:02 UTC
Created attachment 164632 [details] [review]
first attempt

This works for me in GNOME 2.28 and 2.30. I'm still waiting for jhbuild to finish building so I can test it in a 3.0 environment.

Not sure if this is all we need to do or not.
Comment 2 Joanmarie Diggs (IRC: joanie) 2010-06-25 16:40:20 UTC
Created attachment 164636 [details] [review]
take 2

Put back something I ripped out in the first attempt. I thought it was added back when we thought gnome-speech wouldn't be deprecated. Based on Mike Gorse's testing, it looks like it might be there for at-spi over dbus. Oops. Thanks Mike!
Comment 3 Joanmarie Diggs (IRC: joanie) 2010-06-25 16:47:43 UTC
I still need to test that this change works in a 3.0 environment (jhbuild is still building and building....)

In the meantime, does this solve the problem of the dependency?
Comment 4 Javier Jardón (IRC: jjardon) 2010-06-25 20:59:06 UTC
AFAIK It's enough, but It would be great if you can make the gconf dependency optional too (but this has low priority)
Comment 5 Joanmarie Diggs (IRC: joanie) 2010-06-25 21:13:17 UTC
(In reply to comment #4)
> AFAIK It's enough,

Awesome. If it gets committed immediately after I do the 2.31.4 release, is that soon enough?

> It would be great if you can make the gconf dependency
> optional too (but this has low priority)

Oh yeah, good point.... I just opened bug 622764 for that.
Comment 6 Javier Jardón (IRC: jjardon) 2010-06-25 21:55:00 UTC
(In reply to comment #5)
> (In reply to comment #4)
> > AFAIK It's enough,
> 
> Awesome. If it gets committed immediately after I do the 2.31.4 release, is
> that soon enough?
> 

Perfect

> > It would be great if you can make the gconf dependency
> > optional too (but this has low priority)
> 
> Oh yeah, good point.... I just opened bug 622764 for that.

Thanks a lot for take care of this ;)
Comment 8 Joanmarie Diggs (IRC: joanie) 2010-06-28 20:09:46 UTC
Javier, since I'm still a jhbuild n00b.... When you get a chance would you mind making the necessary adjustment to jhbuild moduleset? Thanks!
Comment 9 Javier Jardón (IRC: jjardon) 2010-06-28 20:47:05 UTC
Sure, done here:
http://git.gnome.org/browse/jhbuild/commit/?id=8a465930dad1e6203401a85f8f5ea62ee1ea5d2e

Thanks a lot Joanmarie