Review Request: Make DBusMenuQt optional

[Thiago Macieira]

On Thursday 5. August 2010 15.00.41 Ingo Klöcker wrote:
> On Wednesday 04 August 2010, Marco Martin wrote:
> > On Wednesday 04 August 2010, Marc Mutz wrote:
> > > > On 2010-08-04 16:08:34, Aurélien Gâteau wrote:
> > > > > The change looks OK, but I am wondering why you do not want to
> > > > > link with DBusMenuQt. Having (very) different code paths is
> > > > > potentially a source of trouble for maintenance.
> > > 
> > > Having that kind of library as a hard dependency of KDE is just
> > > gross. Have a look at what other libraries kdelibs depends on, and
> > > how many of them are actually required. This library is used for a
> > > one-liner, literally. It
> > 
> > however there are some bugs in the systray that simply can't be
> > solved without dbusmenu
> 
> Which is completely irrelevant for the usage on a phone.

On the contrary, it's very relevant. The systray is redefined as a set of known 
indicators and the communication must not happen via XEmbed. Any menus added 
to those entries must be communicated to the host application.

-- 
Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org
  Senior Product Manager - Nokia, Qt Development Frameworks
      PGP/GPG: 0x6EF45358; fingerprint:
      E067 918B B660 DBD1 105C  966C 33F5 F005 6EF4 5358
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 190 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20100805/58006d0e/attachment.sig>