diff options
author | Craig Scott <craig.scott@qt.io> | 2021-09-02 14:40:46 +1000 |
---|---|---|
committer | Craig Scott <craig.scott@qt.io> | 2021-09-02 20:52:22 +1000 |
commit | f266381b1ba427485e444640fcd6fb3f68a2b1bb (patch) | |
tree | 1b0cf63c1b85a0f9e8271ad456023931b826ae98 /src/corelib/Qt6CoreMacros.cmake | |
parent | 035dc537bee26e3b63a211b2835d8560439e161f (diff) |
Clean up output target handling left behind by earlier changes
Previous work removed the no-longer-used code path that could add
targets from qml compiler features. The variable that code path would
populate was not fully cleaned up in that removal. Clean it up now and
move the variables related to output target handling closer to the
only remaining area where they could be generated.
Also ensure that we set the variable named by OUTPUT_TARGETS even if
isBinary ends up being true. We should explicitly set that variable
to an empty string in that case.
Amends 363df5cd709762c3c014132c1670fdd715233620
Pick-to: 6.2
Change-Id: I17de9788b390aac9a7aedb29ab9bdfee99c305af
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
Diffstat (limited to 'src/corelib/Qt6CoreMacros.cmake')
-rw-r--r-- | src/corelib/Qt6CoreMacros.cmake | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/src/corelib/Qt6CoreMacros.cmake b/src/corelib/Qt6CoreMacros.cmake index 567d5f2475..508b58f7ae 100644 --- a/src/corelib/Qt6CoreMacros.cmake +++ b/src/corelib/Qt6CoreMacros.cmake @@ -1687,7 +1687,6 @@ function(_qt_internal_process_resource target resourceName) string(REPLACE "/" "_" resourceName ${resourceName}) string(REPLACE "." "_" resourceName ${resourceName}) - set(output_targets "") set(resource_files ${rcc_FILES}) if(NOT "${rcc_BASE}" STREQUAL "") get_filename_component(abs_base "${rcc_BASE}" ABSOLUTE) @@ -1710,11 +1709,10 @@ function(_qt_internal_process_resource target resourceName) if (NOT resource_files) if (rcc_OUTPUT_TARGETS) - set(${rcc_OUTPUT_TARGETS} "${output_target_quick}" PARENT_SCOPE) + set(${rcc_OUTPUT_TARGETS} "" PARENT_SCOPE) endif() return() endif() - list(APPEND output_targets ${output_target_quick}) set(generatedBaseName "${resourceName}") set(generatedResourceFile "${CMAKE_CURRENT_BINARY_DIR}/.rcc/${generatedBaseName}.qrc") @@ -1809,6 +1807,7 @@ function(_qt_internal_process_resource target resourceName) COMMENT "Running rcc for resource ${resourceName}" VERBATIM) + set(output_targets "") if(isBinary) # Add generated .rcc target to 'all' set add_custom_target(binary_resource_${generatedBaseName} ALL DEPENDS "${generatedOutfile}") @@ -1816,11 +1815,10 @@ function(_qt_internal_process_resource target resourceName) set_property(SOURCE "${generatedOutfile}" PROPERTY SKIP_AUTOGEN ON) set_property(TARGET ${target} APPEND PROPERTY _qt_generated_qrc_files "${generatedResourceFile}") - __qt_propagate_generated_resource(${target} ${resourceName} "${generatedOutfile}" output_target) - list(APPEND output_targets ${output_target}) - if (rcc_OUTPUT_TARGETS) - set(${rcc_OUTPUT_TARGETS} "${output_targets}" PARENT_SCOPE) - endif() + __qt_propagate_generated_resource(${target} ${resourceName} "${generatedOutfile}" output_targets) + endif() + if (rcc_OUTPUT_TARGETS) + set(${rcc_OUTPUT_TARGETS} "${output_targets}" PARENT_SCOPE) endif() endfunction() |