expose more flags

Use if DEFINED to make sure the flag still setup when it's defined and value to NO

IoT.js-DCO-1.0-Signed-off-by: Yonggang Luo luoyonggang@gmail.com

Author: lygstate

CMakeLists.txt

@@ -55,6 +55,16 @@ if(NOT DEFINED ENABLE_LTO)
   set(ENABLE_LTO OFF)
 endif()
 
+if(NOT DEFINED JERRY_LINE_INFO)
+  if("${CMAKE_BUILD_TYPE}" STREQUAL "Debug")
+    message("Line info default enabled in debug mode")
+    set(JERRY_LINE_INFO ON)
+  else()
+    message("Line info default disabled in non debug mode")
+    set(JERRY_LINE_INFO OFF)
+  endif()
+endif()
+
 macro(iotjs_add_flags VAR)
   foreach(_flag ${ARGN})
     set(${VAR} "${${VAR}} ${_flag}")

cmake/jerry.cmake

@@ -70,7 +70,7 @@ set_property(TARGET jerry-snapshot PROPERTY
 
 # Utility method to add -D<KEY>=<KEY_Value>
 macro(add_cmake_arg TARGET_ARG KEY)
-  if(${KEY})
+  if(DEFINED ${KEY})
     list(APPEND ${TARGET_ARG} -D${KEY}=${${KEY}})
   endif()
 endmacro(add_cmake_arg)
@@ -112,7 +112,7 @@ endif()
 
 # Add a few cmake options based on buildtype/external cmake defines
 if("${CMAKE_BUILD_TYPE}" STREQUAL "Debug")
-  list(APPEND DEPS_LIB_JERRY_ARGS -DJERRY_ERROR_MESSAGES=ON)
+  set(JERRY_ERROR_MESSAGES ON)
 endif()
 
 # NuttX is not using the default port implementation of JerryScript
@@ -174,8 +174,6 @@ ExternalProject_Add(libjerry
     -DJERRY_LOGGING=ON
     -DJERRY_LINE_INFO=${JERRY_LINE_INFO}
     -DJERRY_VM_EXEC_STOP=ON
-    -DJERRY_ERROR_MESSAGES=ON
-    -DENABLE_LTO=${ENABLE_LTO}
     ${DEPS_LIB_JERRY_ARGS}
     ${EXTRA_JERRY_CMAKE_PARAMS}
   BUILD_BYPRODUCTS ${JERRY_LIB_BUILD_BYPRODUCTS}