<div dir="ltr"><div><div>Let me ask two things regarding dbus activation:<br><br>- Why do we need dbus activation if we now have autostart?<br></div>- Why do we need dbus activation if it does not work[1] from QDBusAbstactInterface, which we use everywhere?<br><br>[1] <a href="https://bugreports.qt-project.org/browse/QTBUG-42891">https://bugreports.qt-project.org/browse/QTBUG-42891</a><br><br></div>Albert<br><div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 9, 2014 at 1:35 PM, Àlex Fiestas <span dir="ltr"><<a href="mailto:afiestas@kde.org" target="_blank">afiestas@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Git commit 408533f25ccaa1c55e1bbfdbc15abe3120736bd5 by Àlex Fiestas.<br>
Committed on 09/12/2014 at 21:27.<br>
Pushed by afiestas into branch 'frameworks'.<br>
<br>
Fix dbus activation by using configure_file + install<br>
<br>
Instead of using CMake build in macros, install and configure the<br>
service file the same way we do in many others KDE projects such ktp or<br>
bluedevil.<br>
<br>
This fixes the dbus activation in different ways:<br>
-Makes it possible to install in different prefixes (not only /usr/lib)<br>
-Replaces the binary with the correct path (taking into account prefix)<br>
<br>
M  +5    -2    kded/CMakeLists.txt<br>
M  +1    -1    kded/<a href="http://org.kde.kdeconnect.service.in" target="_blank">org.kde.kdeconnect.service.in</a><br>
<br>
<a href="http://commits.kde.org/kdeconnect-kde/408533f25ccaa1c55e1bbfdbc15abe3120736bd5" target="_blank">http://commits.kde.org/kdeconnect-kde/408533f25ccaa1c55e1bbfdbc15abe3120736bd5</a><br>
<br>
diff --git a/kded/CMakeLists.txt b/kded/CMakeLists.txt<br>
index 98304e4..46d25a9 100644<br>
--- a/kded/CMakeLists.txt<br>
+++ b/kded/CMakeLists.txt<br>
@@ -23,5 +23,8 @@ endif()<br>
<br>
 install(TARGETS kdeconnectd DESTINATION ${LIBEXEC_INSTALL_DIR})<br>
<br>
-include(DbusActivationMacros)<br>
-dbus_add_activation_service(<a href="http://org.kde.kdeconnect.service.in" target="_blank">org.kde.kdeconnect.service.in</a>)<br>
\ No newline at end of file<br>
+configure_file(<a href="http://org.kde.kdeconnect.service.in" target="_blank">org.kde.kdeconnect.service.in</a><br>
+               ${CMAKE_CURRENT_BINARY_DIR}/org.kde.kdeconnect.service)<br>
+<br>
+install(FILES ${CMAKE_CURRENT_BINARY_DIR}/org.kde.kdeconnect.service<br>
+    DESTINATION ${DBUS_SERVICES_INSTALL_DIR})<br>
diff --git a/kded/<a href="http://org.kde.kdeconnect.service.in" target="_blank">org.kde.kdeconnect.service.in</a> b/kded/<a href="http://org.kde.kdeconnect.service.in" target="_blank">org.kde.kdeconnect.service.in</a><br>
index 847bd1a..e5f5dfd 100644<br>
--- a/kded/<a href="http://org.kde.kdeconnect.service.in" target="_blank">org.kde.kdeconnect.service.in</a><br>
+++ b/kded/<a href="http://org.kde.kdeconnect.service.in" target="_blank">org.kde.kdeconnect.service.in</a><br>
@@ -1,3 +1,3 @@<br>
 [D-BUS Service]<br>
 Name=org.kde.kdeconnect<br>
-Exec=@CMAKE_INSTALL_FULL_LIBEXECDIR@/kdeconnectd<br>
+Exec=${CMAKE_INSTALL_PREFIX}/${LIBEXEC_INSTALL_DIR}/kdeconnectd<br>
<br>
</blockquote></div><br></div></div></div></div></div>