Review Request 125116: Fix install path

Aleix Pol Gonzalez aleixpol at kde.org
Wed Sep 9 17:51:39 UTC 2015


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


+1

Similar patches have happened on most KDE projects already.

- Aleix Pol Gonzalez


On Sept. 9, 2015, 3:58 p.m., Giuseppe Calà wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125116/
> -----------------------------------------------------------
> 
> (Updated Sept. 9, 2015, 3:58 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: kwallet-pam
> 
> 
> Description
> -------
> 
> Patch fixing issue with library's installation path
> 
> 
> Diffs
> -----
> 
>   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/36b5a906/attachment.html>


More information about the Kde-frameworks-devel mailing list