Hello, buildsystem-ers! ;)<br>
<br>
As stated in CMake's doc, INSTALL_FILES command is deprecated in<br>
favour of INSTALL( FILES ... DESTINATION ...) command.<br>
<br>
So the following code<br>
<br>
INSTALL_FILES( ${DATA_INSTALL_DIR}/some_dir files_to_install )<br>
<br>
should be replaced with<br>
<br>
INSTALL( FILES files_to_install DESTINATION ${DATA_INSTALL_DIR}/some_dir )<br>
<br>
Well, we have a problem here:<br>
<br>
${DATA_INSTALL_DIR} contains &quot;/share/apps&quot;.<br>
<br>
It is fine as long as INSTALL_FILES is used, but INSTALL works<br>
correctly only if its DESTINATION argument _doesn't_ begin with &quot;/&quot;<br>
(i.e. it must be share/apps), or it will try to install to root<br>
directory &quot;/share/apps&quot;.<br>
I.e., if INSTALL command sees a &quot;/&quot; in dest-path, it won't prepend<br>
${CMAKE_INSTALL_PREFIX} to it.<br>
<br>
The question is:<br>
Is it planned to replace deprecated INSTALL_FILES with INSTALL in<br>
CMakeLists.txt?<br>
If yes, then ${DATA_INSTALL_DIR} needs to be modified to not include<br>
the leading &quot;/&quot;.<br>
<br>
Rather global change :).<br>
<br>
Thoughts?<br>
<br>
Dmitry.<br>