D5615: fix include path in KF5PackageMacros.cmake

Aleix Pol Gonzalez noreply at phabricator.kde.org
Thu Apr 27 10:46:11 UTC 2017


apol added inline comments.

INLINE COMMENTS

> KF5PackageMacros.cmake:4
> +set(CMAKE_MODULE_PATH ${ECM_KDE_MODULE_DIR})
>  include(KDEInstallDirs)
>  

How about `include(${ECM_KDE_MODULE_DIR}/KDEInstallDirs)`?
This way we don't mess with CMAKE_MODULE_PATH.

REPOSITORY
  R290 KPackage

REVISION DETAIL
  https://phabricator.kde.org/D5615

To: jriddell, #frameworks, mart
Cc: apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170427/d95adefb/attachment.html>


More information about the Kde-frameworks-devel mailing list