-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120080/
-----------------------------------------------------------

Review request for kde-workspace.


Bugs: 329094
    https://bugs.kde.org/show_bug.cgi?id=329094


Repository: khotkeys


Description
-------

The message that qdbusviewer failed to run doesn't always show. This happens 
when qtchooser is installed, but none of the KDE4 or KDE5 qdbusviewers are 
installed, causing qtchooser to end with an error code of 1 and printing an 
error that it can't find qdbusviewer. This patch works around that by using 
KProcess::execute which returns an error code as opposed to just a boolean.


Diffs
-----

  kcm_hotkeys/actions/dbus_action_widget.cpp 
5c72762c60ff74850a6a549db75e759dc4aa86e7 

Diff: https://git.reviewboard.kde.org/r/120080/diff/


Testing
-------

Tried with qdbusviewer both properly installed and only using the qtchooser 
version.


Thanks,

Mathias Tillman

Reply via email to