[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:       Aurélien Gâteau <aurelien.gateau () canonical ! com>
Date:       2010-05-03 14:55:52
Message-ID: 4BDEE3F8.7050402 () canonical ! com
[Download RAW message or body]

On 30/04/2010 14:07, Sebastian Kügler wrote:
> 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).

It is for knetworkmanager. Here is the patch shipped in Ubuntu Lucid.

Aurélien


["kubuntu_03_no_dbusmenu.diff" (text/x-diff)]

diff --git a/monolithic/main.cpp b/monolithic/main.cpp
index eef81f8..d744871 100644
--- a/monolithic/main.cpp
+++ b/monolithic/main.cpp
@@ -17,6 +17,7 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
+#include <stdlib.h>
 
 #include <KAboutData>
 #include <KCmdLineArgs>
@@ -30,6 +31,10 @@ static const char version[] = "v0.9";
 
 int main( int argc, char** argv )
 {
+    // Super ugly hack: DBusMenu can't represent the custom menu items
+    // KNetworkManager creates, so let's disable it for now
+    setenv("KSNI_NO_DBUSMENU", "1", 1 /* overwrite */);
+
     KAboutData about("knetworkmanager", 0, ki18n("KNetworkManager"), version, \
ki18n(description), KAboutData::License_GPL, ki18n("(C) 2009 Will Stephenson"), \
                KLocalizedString(), \
                "http://techbase.kde.org/Projects/Network_Management");
     about.addAuthor( ki18n("Will Stephenson"), ki18n("Original Author, Maintainer"), \
                "wstephenson@kde.org" );
     about.addAuthor( ki18n("Paul Marchouk"), ki18n("User Interface Polish"), \
"pmarchouk@gmail.com" );



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