Gracefully exit if pclint not found

ament · May 1, 2018 · e293644 · e293644
1 parent 5e5e6b1
commit e293644
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/ament_cmake_pclint/CMakeLists.txt b/ament_cmake_pclint/CMakeLists.txt
@@ -1,4 +1,4 @@
-cmake_minimum_required(VERSION 3.5)
+cmake_minimum_required(VERSION 3.9)
 
 project(ament_cmake_pclint NONE)
 

diff --git a/ament_cmake_pclint/cmake/ament_pclint.cmake b/ament_cmake_pclint/cmake/ament_pclint.cmake
@@ -43,7 +43,9 @@ function(ament_pclint)
 
   find_program(ament_pclint_BIN NAMES "ament_pclint")
   if(NOT ament_pclint_BIN)
-    message(FATAL_ERROR "ament_pclint() could not find program 'ament_pclint'")
+    message(WARNING "ament_pclint() could not find program 'ament_pclint', skipping.")
+    set(_ament_pclint_BIN_FOUND "FALSE" PARENT_SCOPE)
+    return()
   endif()
 
   set(cmd "${ament_pclint_BIN}" ${ARG_UNPARSED_ARGUMENTS})
@@ -109,4 +111,10 @@ function(ament_pclint)
     PROPERTIES
     LABELS "pclint;linter"
   )
+  if(NOT _ament_pclint_BIN_FOUND)
+    set_test_properties(
+      "${ARG_TESTNAME}"
+      PROPERTIES
+      DISABLED "TRUE"
+  endif()
 endfunction()