cmake: Add check MAVLINK_DIALECT value

Fix #139.
mavlink · Aug 28, 2014 · edb2c49 · edb2c49 · TSC21 · Aug 28, 2014
1 parent 0e3e071
commit edb2c49
Showing 1 changed file with 14 additions and 4 deletions.
diff --git a/mavros/cmake/Modules/MavrosMavlink.cmake b/mavros/cmake/Modules/MavrosMavlink.cmake
@@ -4,19 +4,29 @@
 find_package(mavlink REQUIRED)
 
 # fallback for older mavlink package.
-if(NOT DEFINED mavlink_DIALECTS)
+if (NOT DEFINED mavlink_DIALECTS)
   list(APPEND mavlink_DIALECTS "ardupilotmega")
   list(APPEND mavlink_DIALECTS "pixhawk")
   list(APPEND mavlink_DIALECTS "common")
-endif()
+endif ()
 
 # Select MAVLink dialect
 set(MAVLINK_DIALECT "ardupilotmega" CACHE STRING "MAVLink dialect selector")
 set_property(CACHE MAVLINK_DIALECT PROPERTY STRINGS ${mavlink_DIALECTS})
 
-# TODO: check that selected dialect are known
+# check that selected dialect are known
+set(MAVLINK_DIALECT_KNOWN)
+foreach (dialect ${mavlink_DIALECTS})
+  if (MAVLINK_DIALECT STREQUAL dialect)
+    set(MAVLINK_DIALECT_KNOWN TRUE)
+  endif ()
+endforeach ()
 
-message(STATUS "Selected MAVLink dialect: ${MAVLINK_DIALECT}")
+if (MAVLINK_DIALECT_KNOWN)
+  message(STATUS "Selected MAVLink dialect: ${MAVLINK_DIALECT}")
+else ()
+  message(FATAL_ERROR "Unknown MAVLink dialect: ${MAVLINK_DIALECT}, known dialects: ${mavlink_DIALECTS}")
+endif ()
 
 # define dialect for mavconn_mavlink.h
 add_definitions(