This is inline with other install statements
Details
Details
Diff Detail
Diff Detail
- Repository
- R290 KPackage
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
While myself I have never made use of it, the idea of using a relative path & not an absolute path with the DESTINATION argument is to allow overriding the install prefix at install time. Or in words of https://cmake.org/cmake/help/v3.0/command/install.html:
If a relative path is given it is interpreted relative to the value of the CMAKE_INSTALL_PREFIX variable. The prefix can be relocated at install time using the DESTDIR mechanism explained in the CMAKE_INSTALL_PREFIX variable documentation.
So given all the other install macros I have seen allow to do that, let's be consistent here and use the relative path as well. I am not aware of any need to use the absolute path here directly.
Not tested though due to broken setup currently, so just +1 and no shipit :)