Review Request 125116: Fix install path

Giuseppe Calà gcala at mail.com
Wed Sep 9 13:58:14 UTC 2015


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

(Updated Set. 9, 2015, 1:58 p.m.)


Review request for KDE Frameworks.


Changes
-------

added missing include


Repository: kwallet-pam


Description
-------

Patch fixing issue with library's installation path


Diffs (updated)
-----

  CMakeLists.txt bfdcfda 

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


Testing
-------

The current CMakeLists.txt has the result, in Chakra Linux, to install the library in /lib64 wich is wrong. With attached patch it seems to follow right behaviour. Old and new version of CMakeLists.txt were tested with the following command:

cmake -DCMAKE_BUILD_TYPE=RelWithDebInfo \
        -DCMAKE_INSTALL_PREFIX=/usr \
        -DLIB_INSTALL_DIR=lib \
        -DSYSCONF_INSTALL_DIR=/etc \
        -DLIBEXEC_INSTALL_DIR=lib \
        -DUDEV_RULES_INSTALL_DIR=/usr/lib/udev/rules.d \
        -DBUILD_TESTING=OFF \
        -DKDE_INSTALL_USE_QT_SYS_PATHS=ON


Thanks,

Giuseppe Calà

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150909/13824fe2/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list