kde-core-devel@kde.org
[Top] [All Lists]

Re: New dependency for kdelibs and kdebase: dbusmenu-qt

Subject: Re: New dependency for kdelibs and kdebase: dbusmenu-qt
From: Sebastian Kügler
Date: Fri, 30 Apr 2010 14:07:44 +0200
If it's for knetworkmanager, please send it to Will and me so we can include it 
to keep knm working; the plasmoid is a different story (it does do contextmenu 
stuff, but is not a ksni).

"Tom Albers" <toma@xxxxxxx> wrote:

>
>On Thu, 29 Apr 2010 21:28:48 +1200, Ben Cooksley <sourtooth@xxxxxxxxx>
>wrote:
>>> To ensure KNetworkManager was still usable with dbusmenu, I added a
>>> hackish way to disable dbusmenu in KStatusNotifierItem: it won't export
>>> the menu on dbus if you set the environment variable
>"KSNI_NO_DBUSMENU".
>>> It's a dirty hack, but I felt it would not be a good idea to add a
>>> proper API for this, as we should encourage developers to use standard
>>> menu items. Kubuntu Lucid will ship with a one-line patch in
>>> KNetworkManager which sets this environment variable in main(). I
>>> haven't proposed it for merging, but I can do it if you feel it's
>>> necessary.
>> 
>> Please do it. This will ensure other distributions get it so that
>> KNetworkManager isn't broken for users when they update to KDE 4.5.
>> Avoiding breakages to critical ( in the eyes of users anyway ) tools
>> is a must.
>
>fwiw, i agree
>
>-- 
>Tom Albers
>KDE Developer

-- 
Sent from my phone -- please excuse my brevity.
<Prev in Thread] Current Thread [Next in Thread>