Review Request 125843: Use KDE_INSTALL_DBUSINTERFACEDIR to install dbus interfaces
Heiko Becker
heirecka at exherbo.org
Wed Oct 28 20:33:17 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125843/
-----------------------------------------------------------
(Updated Oct. 28, 2015, 8:33 p.m.)
Status
------
This change has been marked as submitted.
Review request for Plasma and Martin Gräßlin.
Changes
-------
Submitted with commit 2c4ed0aca102be6f9a90535ec42b7a102ca7e24b by Heiko Becker to branch master.
Repository: kwin
Description
-------
...and use PATH_VARS to make the config file work with absolute paths.
Two reasons to do this:
- DBUS_INTERFACES_INSTALL_DIR is marked deprecated
- Not hard-coding the packackage prefix is helpful on a multiarch
layout where the prefix is /usr/${host} but arch-independent files
should still be installed to /usr/share (i.e a level below the
prefix).
Diffs
-----
CMakeLists.txt 44a423cb723121537a2c2adaa17c1d9f8875f9b8
KWinDBusInterfaceConfig.cmake.in 762a2ed20221860c50f70a1e551eb74b3b1e0f3a
Diff: https://git.reviewboard.kde.org/r/125843/diff/
Testing
-------
The affected files are still installed in the desired place and the paths in KWinDBusInterfaceConfig.cmake are correct.
Thanks,
Heiko Becker
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151028/5acd383e/attachment.html>
More information about the Plasma-devel
mailing list