On 11/30/2012 12:36 PM, Stephen Kelly wrote:
> Stephen Kelly wrote:
>> Currently, in the add-INTERFACE_LINK_LIBRARIES-property branch, the
>> INTERFACE_LINK_LIBRARIES property is read for static libraries if the
>> policy is new,
> 
> I've changed that now in the branch. Effort is made to not populate the 
> property, and even if it does get populated, the value is not exported.

Great.

>> even though it is LINK_LIBRARIES that we want to read in
>> the case of STATIC libraries. That can only really be addressed when my
>> followup topic to introduce the LINK_LIBRARIES property is merged. The
>> question is how to deal with that?
>>
>> I can update add-INTERFACE_LINK_LIBRARIES-property to skip reading
>> INTERFACE_LINK_LIBRARIES for STATIC libraries

It already does skip it.  The entire modification you made is inside
an if condition:

  if(this->GetType() == cmTarget::SHARED_LIBRARY ||
     this->IsExecutableWithExports())

-Brad
--

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