Brad King wrote: > On 04/25/2013 03:34 AM, Stephen Kelly wrote: >> I haven't had time to investigate fully, but this patch should 'fix' the >> problem: > [snip] >> I'll investigate later to see if it's the right fix and why. > > Great! I've turned that patch into this commit: > > http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=de9d4a63 > > See commit message for an explanation that makes sense to me.
The commit message is correct, but it's not the whole story. We also need to validate the it->Value as a target name acceptable to TARGET_PROPERTY. I've pushed the fix-multi-config-tll-include-dirs branch to my clone. Thanks, Steve. -- 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://www.cmake.org/mailman/listinfo/cmake
