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
-- 

Powered by www.kitware.com

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers

Reply via email to