Eyescale/CMake

Wrong usage of the CMake set function in FindCppcheck.cmake

lguyot opened this issue · 0 comments

lguyot commented

Line 44:

set(CPPCHECK_EXECUTABLE  "notfound" CACHE PATH FORCE "")

should be replaced by

set(CPPCHECK_EXECUTABLE "notfound" CACHE PATH "" FORCE)

see http://www.cmake.org/cmake/help/cmake2.6docs.html#command:set.

Tests (Linux 64, Ubuntu 13.04, CMake 2.8.10.1) show that CPPCHECK_EXECUTABLE can be set to notfound;CACHE;PATH;FORCE instead of being reset to "" in case of an incorrect installation.