[cmake-developers] [PATCH][CPack][RPM] multiple path relocation prefixes

Brad King brad.king at kitware.com
Fri Jan 16 16:38:42 EST 2015

On 01/16/2015 08:42 AM, Brad King wrote:
> Go ahead and merge to 'next' for testing and follow up with any
> dashboard trouble.  I'll still be able to review it there.

AFAICT the change to GNUInstallDirs is just to help projects use
the new feature, and not actually part of the new feature itself.
The feature itself just happens to use the same set of <dir> values.
It feels a bit strange to have CPACK_ variables set by GNUInstallDirs,
and to require CPACK_PACKAGING_INSTALL_PREFIX to be set before that
module is included.

Perhaps instead CPackRPM can provide an option to compute a default
for CPACK_PACKAGING_INSTALL_FULL_<dir> based on the GNUInstallDirs
values.  This way GNUInstallDirs does not have to change at all,
and users can choose to include GNUInstallDirs and still set their
own CPACK_PACKAGING_INSTALL_FULL_<dir> values separately.


Typically I quote the second argument in this case.  It is not
strictly necessary but looks cleaner IMO.

Please look at adding test cases, perhaps in Tests/RunCMake,
to check the warning message about non-relocatable paths.


More information about the cmake-developers mailing list