diff options
author | Joerg Bornemann <joerg.bornemann@qt.io> | 2021-03-01 17:13:56 +0100 |
---|---|---|
committer | Joerg Bornemann <joerg.bornemann@qt.io> | 2021-03-09 16:57:03 +0100 |
commit | c4f5762b20dc20bab3cc62e9166d0e5b36e21cc6 (patch) | |
tree | 531ca2ae4d9c5220d44c68af12c3a867c22f7d9d /cmake/QtFeature.cmake | |
parent | 1d567eb63d38683c23c29b618430b7d1d4d49b4d (diff) |
configure: Fix error when turning on/off non-emitted features
Features that are not emitted in the current
configuration (e.g. plugin-manifests on Linux) should not yield an
error. Instead, print a warning message like Qt5's configure did.
Set insignificant feature to OFF.
Remove the now unneeded emit_if parameter from
qt_feature_set_cache_value.
Pick-to: 6.1
Fixes: QTBUG-88305
Change-Id: I0f2ce152fca5f08417038c9bd2c07639ff6a3df4
Reviewed-by: Alexey Edelev <alexey.edelev@qt.io>
Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
Diffstat (limited to 'cmake/QtFeature.cmake')
-rw-r--r-- | cmake/QtFeature.cmake | 28 |
1 files changed, 14 insertions, 14 deletions
diff --git a/cmake/QtFeature.cmake b/cmake/QtFeature.cmake index 6e8a653c19..75a9d91560 100644 --- a/cmake/QtFeature.cmake +++ b/cmake/QtFeature.cmake @@ -227,13 +227,8 @@ function(_qt_internal_dump_expression_values expression_dump expression) set(${expression_dump} "${${expression_dump}}" PARENT_SCOPE) endfunction() -function(qt_feature_set_cache_value resultVar feature emit_if condition calculated label) +function(qt_feature_set_cache_value resultVar feature condition calculated label) if (DEFINED "FEATURE_${feature}") - # Must set up the cache - if (NOT (emit_if)) - message(FATAL_ERROR "Sanity check failed: FEATURE_${feature} that was not emitted was found in the CMakeCache.") - endif() - # Revisit value: set(cache "${FEATURE_${feature}}") @@ -257,12 +252,8 @@ meet its condition after reconfiguration.") set("FEATURE_${feature}" "${cache}" CACHE BOOL "${label}" FORCE) else() # Initial setup: - if (emit_if) - set("FEATURE_${feature}" "${calculated}" CACHE BOOL "${label}") - set(result "${calculated}") - else() - set(result OFF) - endif() + set("FEATURE_${feature}" "${calculated}" CACHE BOOL "${label}") + set(result "${calculated}") endif() set("${resultVar}" "${result}" PARENT_SCOPE) @@ -350,8 +341,17 @@ function(qt_evaluate_feature feature) endif() endif() - qt_feature_set_cache_value(cache "${feature}" "${emit_if}" "${condition}" "${result}" - "${arg_LABEL}") + if(NOT emit_if AND DEFINED FEATURE_${feature}) + set(msg "") + string(APPEND msg + "Feature ${feature} is insignificant in this configuration, " + "ignoring related command line option(s).") + qt_configure_add_report_entry(TYPE WARNING MESSAGE "${msg}") + set(result OFF) + set(FEATURE_${feature} OFF) + endif() + + qt_feature_set_cache_value(cache "${feature}" "${condition}" "${result}" "${arg_LABEL}") qt_feature_set_value("${feature}" "${cache}" "${condition}" "${arg_LABEL}" "${arg_CONDITION}") |