On 11/27/2013 10:40 AM, Brad King wrote:
> Go ahead and merge it to 'next' for testing.  I will review it
> there when I return from vacation next week.

Please extend the topic with updates to the test suite to cover the
new option.  Even if it is too hard to test that the independent
build behavior works we at least need to verify that using the
option does not break anything.

Also please add a RunCMake.ExternalProject test with a case to
verify that this message appears:

+        message(AUTHOR_WARNING "Using NO_DEPENDS for \"${step}\" step  might 
break parallel builds")

Also remove extra indentation from the above-quoted line.

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