Improved export-sets in CMake

Stephen Kelly steveire at gmail.com
Mon Jun 4 21:35:46 UTC 2012


Stephen Kelly wrote:

> Yury G.  Kudryashov <urkud.urkud <at> gmail.com> writes:
>> > 
>> > After that I'd post to the cmake list for initial feedback.
>> `git bisect` tells me that 'Remove cmTargetExport constructor'
>> (07901863b6) makes test #53 (ExportImport) fail. I fail to understand
>> why.
> 
> I looked into this, and the problem is that you do not initialize the
> FrameworkGenerator member, and the class has no default ctor to initialize
> it either. You need a line like:
> 
>       te->FrameworkGenerator = frameworkGenerator;

Ah, looks like you figured that out last month.

> 
> Please also add a test or extend the existing test. Also, why is the
> content of 9bbb00ef9214aef75152f4b8c6147294a57b2103 in a separate commit?
> It would make sense to add that stuff in the commit that adds the class.
> 
> Do you have time to finish this feature?

Thanks,

Steve.




More information about the Kde-buildsystem mailing list