Fix "error activiting kdeconnectd" for kdeconnect-cli

Authored by mtijink on Nov 29 2017, 8:02 PM.



Kdeconnect-cli always displayed one or more "error activating kdeconnectd: QDBusError("", "")" messages.

This diff uses the reply of starting the kdeconnect daemon correctly, so this does not happen anymore.

Test Plan

Kdeconnect-cli and the daemon still work, but kdeconnect-cli does not show the error message anymore. I verified that if, for some reason, the daemon could not start, it does display an error.

Diff Detail

R224 KDE Connect
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
mtijink requested review of this revision.Nov 29 2017, 8:02 PM
mtijink created this revision.
mtijink edited the summary of this revision. (Show Details)Nov 29 2017, 8:05 PM
mtijink edited the test plan for this revision. (Show Details)
mtijink added a reviewer: KDE Connect.
apol accepted this revision.Nov 30 2017, 2:42 PM
apol added a subscriber: apol.


This revision is now accepted and ready to land.Nov 30 2017, 2:42 PM
This revision was automatically updated to reflect the committed changes.