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

Reply via email to