From f266381b1ba427485e444640fcd6fb3f68a2b1bb Mon Sep 17 00:00:00 2001 From: Craig Scott Date: Thu, 2 Sep 2021 14:40:46 +1000 Subject: [PATCH] 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 --- src/corelib/Qt6CoreMacros.cmake | 14 ++++++-------- 1 file 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()