diff options
author | Stephen Kelly <steveire@gmail.com> | 2016-03-21 22:24:56 +0100 |
---|---|---|
committer | Stephen Kelly <steveire@gmail.com> | 2016-03-21 22:31:09 +0100 |
commit | 928e79cf5d51a3c78d388b7eb972b3dae1ebbb68 (patch) | |
tree | 9ca1c5c89cb6b8127baaf1827337cd8c1b1ef7e4 /modules/ECMGeneratePriFile.cmake | |
parent | 09008700af42967f6ad10c9b3ef565cc2f727874 (diff) | |
download | extra-cmake-modules-928e79cf5d51a3c78d388b7eb972b3dae1ebbb68.tar.gz extra-cmake-modules-928e79cf5d51a3c78d388b7eb972b3dae1ebbb68.tar.bz2 |
Fix _ecm_update_iconcache to only update the install location
Use the CMAKE_INSTALL_PREFIX to ensure that only a file in the install
location will be touched. Otherwise a user running
make install
as root would create root-owned files in the build directory.
Use the cmake -E touch command to update the mtime of the directory.
That command exists since late 2007, but this manual file creation
workaround pre-dates that by a few months (kdelibs SVN revision 691973).
There is no chance any calling code is already passing an absolute path
here because later in the function it is already used together with a
CMAKE_INSTALL_PREFIX.
Inspired-by: REVIEW 127445
Diffstat (limited to 'modules/ECMGeneratePriFile.cmake')
0 files changed, 0 insertions, 0 deletions