On 11/06/2012 08:55 AM, Stephen Kelly wrote:
> + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wnon-virtual-dtor -ansi -Wcast-
> align -Wchar-subscripts -Wall -W -Wshadow")
Adding "-ansi" here will require several other fixes as
revealed by the dashboard last night.
Also, your fix commit:
http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=853c22b2
added use of cmTarget::UNKNOWN_TARGET which does not exist.
It is UNKNOWN_LIBRARY.
Finally, existing dashboard scripts already add many of these
flags. Now we're duplicating them. They should be added in
a loop that adds each one only if not already present.
-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
