Review Request 129552: Fix dbus-activation

Albert Vaca Cintora albertvaka at gmail.com
Thu Nov 24 12:51:03 UTC 2016


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

(Updated nov. 24, 2016, 4:51 a.m.)


Review request for Extra Cmake Modules, David Faure and Hannah von Reth.


Changes
-------

Oops, old diff.


Repository: extra-cmake-modules


Description
-------

For dbus-activation to work, .service files have to be installed in /share/dbus-1/services, as there's where dbus-daemon looks for them.

Our approach of installing everything in /bin/data under Windows works for Qt, but not for other libs and daemons, which still look for stuff in /share.

An alternative solution would be to patch dbus-daemon and everything else, but I think this is cleaner than having to maintain an increasing number of patches for external projects.


Diffs (updated)
-----

  kde-modules/KDEInstallDirs.cmake 7f5e44778b85906d3a619d0ce2c15cd0ead0eaac 

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


Testing
-------

KDE Connect now works :D


Thanks,

Albert Vaca Cintora

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20161124/7b5fe1e7/attachment.html>


More information about the Kde-buildsystem mailing list