D22626: KAlarm: simplify the starting of KOrganizer to use DBus activation

[David Faure]

dfaure added a comment.


  You need to update korganizer in order to grab this recent commit: https://cgit.kde.org/korganizer.git/commit/?id=80a7965ae6a22628c08a2db6c7cc6c913a48605f
  This is what makes DBus activation possible.

INLINE COMMENTS

> djarvie wrote in functions.cpp:1855
> Please leave formatting unchanged.

The `&&` condition on the second line being aligned with `if` and not with the first condition `(!reply...`, was on purpose? Quite unusual, but OK, your call.

REVISION DETAIL
  https://phabricator.kde.org/D22626

To: dfaure, djarvie, vkrause
Cc: kde-pim
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20190722/0a98643e/attachment.html>