Merge lp:~michihenning/unity-api/add-check-for-cppcheck into lp:unity-api

[Michi Henning]

Added warning if cppcheck isn't found.

Added warning if cppcheck isn't found.
Fixes https://bugs.launchpad.net/unity-api/+bug/1190374

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:56
http://jenkins.qa.ubuntu.com/job/unity-api-ci/67/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-api-saucy-armhf-ci/24
        deb: http://jenkins.qa.ubuntu.com/job/unity-api-saucy-armhf-ci/24/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-api-saucy-i386-ci/25

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-api-ci/67/rebuild

review: Approve (continuous-integration)

[Jussi Pakkanen (jpakkane)]

Same comment as with unity-scopes-api.

review: Approve

=== modified file 'CMakeLists.txt'
--- CMakeLists.txt	2013-06-10 23:56:09 +0000
+++ CMakeLists.txt	2013-06-12 21:37:27 +0000
@@ -46,14 +46,18 @@
 
 # Static C++ checks
 
-set(cppcheck_flags --check-config --inline-suppr --enable=all -q --error-exitcode=2)
-set(cppcheck_flags ${cppcheck_flags} --template '{file}({line}): {severity} ({id}): {message}')
-
-add_custom_target(cppcheck COMMAND cppcheck ${cppcheck_flags}
-    ${CMAKE_SOURCE_DIR}/src
-    ${CMAKE_SOURCE_DIR}/test
-    ${CMAKE_BINARY_DIR}/test
-)
+find_program(CPPCHECK_COMMAND NAMES cppcheck)
+if (CPPCHECK_COMMAND)
+    set(CPPCHECK_COMMAND_OPTIONS --check-config --inline-suppr --enable=all -q --error-exitcode=2)
+    set(CPPCHECK_COMMAND_OPTIONS ${CPPCHECK_COMMAND_OPTIONS} --template '{file}({line}): {severity} ({id}): {message}')
+    add_custom_target(cppcheck COMMAND ${CPPCHECK_COMMAND} ${CPPCHECK_COMMAND_OPTIONS}
+        ${CMAKE_SOURCE_DIR}/src
+        ${CMAKE_SOURCE_DIR}/test
+        ${CMAKE_BINARY_DIR}/test
+    )
+else()
+    message(WARNING "Cannot find cppcheck: cppcheck target will not be available")
+endif()
 
 #
 # Definitions for testing with valgrind.