diff options
| author | Craig Scott <craig.scott@qt.io> | 2021-09-02 14:40:46 +1000 |
|---|---|---|
| committer | Qt Cherry-pick Bot <cherrypick_bot@qt-project.org> | 2021-09-02 13:52:34 +0000 |
| commit | bd006b88ea4b292e482f1587fd179dea2fbb5f6a (patch) | |
| tree | 9a66ac5f26a45c38a52802949949ae11536e55a5 /src/corelib/Qt6CoreMacros.cmake | |
| parent | 047eeddf1658ea5bb58d3bb01f93a7a7544852db (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
Change-Id: I17de9788b390aac9a7aedb29ab9bdfee99c305af
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
(cherry picked from commit f266381b1ba427485e444640fcd6fb3f68a2b1bb)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
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 001b6be0ee..cff947d808 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() |