[cmake-developers] Review request: Qt-auto-generators

Stephen Kelly steveire at gmail.com
Thu Oct 24 13:42:44 EDT 2013


Alexander Neundorf wrote:

> On Thursday 24 October 2013, Stephen Kelly wrote:
>> Hi,
>> 
>> I've pushed the Qt-auto-generators topic to stage. This topic was
>> discussed here:
>> 
>>  http://thread.gmane.org/gmane.comp.programming.tools.cmake.devel/7936
>> 
>> I have tested it with ports of KDE Frameworks and GammaRay to use the
>> features.
>> 
>> Alex, could you also review the topic please?
> 
> Looks good in general.
> Just some nitpicking: this is still Qt-specific, so I would prefer to have
> something related to Qt in the names of the target properties and output
> and maybe also if the generated files instead of AUTOGEN_TARGETS_FOLDER,
> "AUTOGEN:" and  and AutogenInfo.cmake.

I thought about that but couldn't think of better, more suitable names.

Also, just because it's Qt-specific now doesn't mean it always will be. If 
there's a reason for something similar relating to another tool, another 
rename would be needed.

I'd prefer to stick with Autogen*.

Thanks,

Steve.





More information about the cmake-developers mailing list