On 07/25/2013 09:16 AM, Stephen Kelly wrote: > target_link_libraries(foo SYSTEM Qt5::Core)
That looks okay to me so long as SYSTEM must always be the first keyword. We could also consider adding it only to the modern (newest) keyword signature to encourage conversion to it. > Should we treat the INTERFACE_INCLUDE_DIRECTORIES of all IMPORTED targets as > SYSTEM includes automatically? I don't think so because one could be importing targets from a dependency that was just built as part of a "superbuild" and may want to see the warnings. -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
