[pkg-wpa-devel] Bug#582793: Bug#582793: wpagui: use knotify to display notifications when running under KDE

[Kel Modderman]

Hi Raphael,

On Monday 24 May 2010 03:54:37 Raphael Geissert wrote:
> Package: wpagui
> Version: 0.6.10-2
> Severity: wishlist
> Tags: patch
> 
> Hi,
> 
> Attached patch makes wpa_gui use KDE's knotify system to display the 
> notifications when a full KDE session is running.

Thanks very much.

> 
> It is implemented by using the kdialog(1) command so that there's no need to 
> link to any KDE-specific library.
> 
> It falls back to QSystemTray's showMessage() if something went wrong when 
> starting kdialog(1).

But it doesn't fallback if kdialog starts but ultimately fails (eq, run as
root -> kdialog(...): Session bus not found). This does:
---
		if (QProcess::execute("/usr/bin/kdialog", args) == 0)
		    return;
	}

	if (!QSystemTrayIcon::supportsMessages())
		return;
---

Thanks, Kel.