On 11/01/2013 09:04 AM, Brad King wrote: > On 11/01/2013 08:49 AM, Brad King wrote: >> On 11/01/2013 07:32 AM, Stephen Kelly wrote: >>> add_library(foo SHARED foo.cpp) >>> add_library(bar SHARED foo.cpp) >>> # target_link_libraries(foo bar) >>> set_property(TARGET foo PROPERTY LINK_LIBRARIES bar) >>> file(GENERATE OUTPUT outputfile >>> CONTENT "$<TARGET_PROPERTY:foo,INTERFACE_LINK_LIBRARIES>\n") [snip] > So yes, hacking the generator expression node for the fallback > is probably the right fix.
On third thought, the tll documentation for the plain signature does not say it populates the INTERFACE_LINK_LIBRARIES property. Therefore I think it is okay for the generator expression that reads it to get an empty value. If a project wants advanced use cases like that it should use the tll keyword signatures or set INTERFACE_LINK_LIBRARIES itself like the tll docs suggest. Therefore I think the fix as of commit 42a17cc6 is correct except that it still needs tests for exporting cases. Thanks, -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
