D15795: Fix warnings about deprecated install dirs
Dino H.
noreply at phabricator.kde.org
Thu Sep 27 16:33:53 BST 2018
dinoh added inline comments.
INLINE COMMENTS
> ltoscano wrote in KDEInstallDirs.cmake:564
> I'm going to comment only on this, but the others (most, if not all) are the same. If you check the definition of _define_relative, this parameter is:
>
> oldstylename (optional): the old-style name of the variable
> ===========================================================
>
> So this command is exactly defining the old variable in addition to the new one.
> We can't remove the definition of the old variables until they are in use, and for sure not during the lifetime of Frameworks 5.x.
>
> Unless of course I misunderstood the code...
Hmm, you're right, but then it shouldn't come up with a warning...?
In the _define_relative macro we have:
if(KDE_INSTALL_${varname})
...
elseif(${_oldstylename})
if(NOT CMAKE_VERSION VERSION_LESS 3.0.0)
message(DEPRECATION "${_oldstylename} is deprecated, use KDE_INSTALL_${varname} instead.")
In my case the message was triggered for all definitions.
REPOSITORY
R240 Extra CMake Modules
REVISION DETAIL
https://phabricator.kde.org/D15795
To: dinoh, apol, bruns, dfaure
Cc: ltoscano, kde-frameworks-devel, kde-buildsystem, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20180927/2b6edbc2/attachment-0001.html>
More information about the Kde-buildsystem
mailing list