On 04/29/2013 06:29 AM, Stephen Kelly wrote:
> I've force-pushed the fix-multi-config-tll-include-dirs branch with a more-
> simple fix for this issue to my clone.

Okay, that looks good.  Please rebase on the partial fix I merged last
week so we can test it in 'next'.  I'll squash all that together later.

Also I realized my local topic using GetLinkInformation is wrong because
it includes usage requirements from everything linked, not just from
the link *interface* closure.  I'll drop that one in favor of yours.

While working through all of the above I realized that AppendTllIncludes
and LinkInterfaceIncludeDirectoriesEntries are not safe.  Users can
re-write the LINK_LIBRARIES property and those other structures will
not be updated.  IIUC they only exist to provide a backtrace for
include directory entries.  Please drop them for now at the expense of
the backtraces which can be restored later by tracking the backtraces
of all tll() link information (similar to tid() now).

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

Reply via email to