On 09/17/2014 02:37 PM, Rolf Eike Beer wrote:
What one actually wants is to write

   if(arg STREQUAL "OPTIONS")

i.e. arg will get expanded, "OPTIONS" not. Changing everything away from
that pattern will in fact make things worse, and will not help you if I
introduce an xOPTION variable now. So, is this warning too noisy or are
we doing something fundamentally wrong?

One obvious case where it probably warns is
        if(arg STREQUAL "WIN32")

Because WIN32 is also a variable with the value "1" on windows this check probably didn't actually do what it was intended to do with the old behavior.

Maybe modules loaded from the cmake installation itself should implicitly have their policy version set to the current cmake version?

Nils
--

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers

Reply via email to