Templates for frameworks CMake files
Stephen Kelly
steveire at gmail.com
Wed Sep 25 20:10:44 UTC 2013
Aurélien Gâteau wrote:
>> ECM and Qt version bumps are not as much an issue as the cmake one. The
>> ECM and Qt version deps are not as likely to change.
>
> I don't understand why you think specifying the cmake version is an issue:
> it should be easy to update the template when this happens, isn't it?
Yes, I suppose.
>> Simplify by putting NO_MODULE before REQUIRED and removing COMPONENTS:
>>
>>find_package(Qt5 ${REQUIRED_QT_VERSION} NO_MODULE REQUIRED Widgets)
>
> Done.
Are you sure? The reviewboard shows otherwise.
>>
>> > if(NOT kdelibs_SOURCE_DIR)
>> > find_package(KF5 5.0.0 REQUIRED MODULE COMPONENTS CMake Compiler
>> > InstallDirs) # Required KF5 frameworks to build this framework
>>
>> I still think this stuff is odd. We're building KF5, yet we need to find
>> KF5 to do so.
>
> This line is only used when building the framework outside of kdelibs, in
> this case we are not building KF5.
We use find_package(KF5) when building each KF5 framework, and when building
the 'monolithic' (but temporary) kdelibs.git.
Thanks,
Steve.
More information about the Kde-frameworks-devel
mailing list