cmake: Bail out in case syncqt failed to run

Make sure to react on failed syncqt runs during the CMake run.

CMake will now report something along this:
```
syncqt couldn't parse .../qt5/qtbase/sync.profile: Can't modify modulus (%) in scalar assignment at .../qt5/qtbase/sync.profile line 25, near ");"
CMake Error at qtbase/cmake/QtModuleHelpers.cmake:192 (message):
  Failed to run syncqt, return code: 255
```

Change-Id: I575a5fc926c547b6b633583e5d675f7e35beb734
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
This commit is contained in:
Kevin Funk 2021-01-15 10:44:42 +01:00
parent 96e9c8cc8a
commit 9ff448b6db

View File

@ -201,7 +201,10 @@ function(qt_internal_add_module target)
-builddir "${PROJECT_BINARY_DIR}" -builddir "${PROJECT_BINARY_DIR}"
"${PROJECT_SOURCE_DIR}") "${PROJECT_SOURCE_DIR}")
message(STATUS "Running syncqt for module: '${module_include_name}' ") message(STATUS "Running syncqt for module: '${module_include_name}' ")
execute_process(COMMAND ${syncqt_full_command}) execute_process(COMMAND ${syncqt_full_command} RESULT_VARIABLE syncqt_ret)
if(NOT syncqt_ret EQUAL 0)
message(FATAL_ERROR "Failed to run syncqt, return code: ${syncqt_ret}")
endif()
set_target_properties("${target}" PROPERTIES set_target_properties("${target}" PROPERTIES
INTERFACE_MODULE_HAS_HEADERS ON INTERFACE_MODULE_HAS_HEADERS ON