[cmake-developers] [PATCH] Do not output transitive dependencies in feature_summary()

Alex Merry kde at randomguy3.me.uk
Mon Feb 3 17:59:01 EST 2014


On 03/02/14 19:27, Stephen Kelly wrote:
> It's generally preferable to write an email and attach the patch.

Noted.

> Alex Merry wrote:
>> +    set_package_properties(${dep} PROPERTIES TRANSITIVE_DEPENDENCY)
> 
> I prefer not to create public API in set_package_properties for something 
> that doesn't need to be public. find_dependency() should be all the public 
> API needed.
> 
> I counter-propose the FeatureSummary-no-transitive topic which I just 
> pushed. What do you think?
> 
>  http://cmake.org/gitweb?p=stage/cmake.git;a=commitdiff;h=a6971f651

I wasn't sure how changes to find_package for something like this would
go down.  This seems like a better solution indeed.

Alex



More information about the cmake-developers mailing list