Review Request 121544: use GNUInstallDirs instead of 'old-school' LIB_SUFFIX & co

[Daniele E. Domenichelli]

> On Dec. 16, 2014, 2:34 p.m., Daniele E. Domenichelli wrote:
> > cmake/modules/TelepathyDefaults.cmake, lines 155-157
> > <https://git.reviewboard.kde.org/r/121544/diff/1/?file=333443#file333443line155>
> >
> >     You should not offer these ``XXX_DESTINATION`` variables to the user. ``CMAKE_INSTALL_XXXDIR`` are already cached, modifying these variables will modify the ``DESTINATION`` ones too.
> >     In this way you will just get extra cmake cached variables in GUIs, and you won't be able to modify them, since they are forced every time.
> >     You can either make them ``INTERNAL`` instead of ``PATH`` (``FORCE`` is implied in this case), or you can just set them as normal variables since you are including the file in the main ``CMakeLists.txt`` (therefore you don't have problems with variable scope), and since it does not make much sense to cache them.
> >     
> >     Also I think I remember that there is some reason to use the **relative** path instead of **absolute** when doing ``install()`` (therefore ``CMAKE_INSTALL_XXXDIR`` instead of ``CMAKE_INSTALL_FULL_XXXDIR``), but I might be wrong here...
> 
> Hrvoje Senjan wrote:
>     > Also I think I remember that there is some reason to use the relative path instead of absolute when doing install() (therefore CMAKE_INSTALL_XXXDIR instead of CMAKE_INSTALL_FULL_XXXDIR), but I might be wrong here...
>     
>     IIRC there was a bug in some cmake version, where INTERFACE target_include_directories couldn't be constructed with *relative* path...
>     and we need the _FULL_ vars due to pc file at least. i know at least some distros are accustomed to kdelibs4-buildsystem where it was perfectly safe to pass absolute path's.

I think that the issue related to using CPack to create an installer and installing files with absolute paths... Nothing really important.


- Daniele E.


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


On Dec. 16, 2014, 6:02 p.m., Hrvoje Senjan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121544/
> -----------------------------------------------------------
> 
> (Updated Dec. 16, 2014, 6:02 p.m.)
> 
> 
> Review request for Telepathy and Aleix Pol Gonzalez.
> 
> 
> Repository: telepathy-logger-qt
> 
> 
> Description
> -------
> 
> this is less fragile in abs. vs. relative paths, etc..
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 0022600 
>   TelepathyLoggerQt/CMakeLists.txt 125b56b 
>   TelepathyLoggerQt/TelepathyLoggerQt.pc.in 508fe25 
>   TelepathyLoggerQt/TelepathyLoggerQtConfig.cmake.in 0a4eef9 
>   cmake/modules/TelepathyDefaults.cmake ec091e2 
> 
> Diff: https://git.reviewboard.kde.org/r/121544/diff/
> 
> 
> Testing
> -------
> 
> so far tried only cmake sets correct vars...
> 
> 
> Thanks,
> 
> Hrvoje Senjan
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20141218/49f8a24c/attachment.html>