[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: New dependency for kdelibs and kdebase: dbusmenu-qt
From:       Sebastian_Kügler <sebas () vizZzion ! org>
Date:       2010-04-30 12:07:44
Message-ID: 4bc5eef2-9e71-4842-8049-431f384402a1 () email ! android ! com
[Download RAW message or body]

[Attachment #2 (text/plain)]

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@kde.org> wrote:

> 
> On Thu, 29 Apr 2010 21:28:48 +1200, Ben Cooksley <sourtooth@gmail.com>
> 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 list] [next in list] [prev in thread] [next in thread]