1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Rework the argument escaping and wrapping in _qt_internal_create_command_script

Browse Source 
Use the quote-based argument wrapping when generating the command line
scripts for the tests. Apply the wrapping and escaping of the arguments
right before writing the script with the following advanced rules:
- if argument is already wrapped using quotes, skip wrapping
- if argument is a CMake variable, skip wrapping
- wrap the argument with qoutes otherwise

The above should cover the most common usecases in centralized place
when we generate the test wrapper scripts.

Remove other ways we wrap arguments in other places, like square bracket
wrapping in qt_internal_create_test_script.

Change-Id: If287dd75d6fb36260b5cf8a687215bda9c9fc1c0
Reviewed-by: Tor Arne Vestbø <tor.arne.vestbo@qt.io>
Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
This commit is contained in:
Alexey Edelev 2024-10-02 13:08:42 +02:00
parent 4d908a3b5e
commit ba09bec5ec
2 changed files with 20 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
cmake/QtPublicTestHelpers.cmake
View File

@ -84,20 +84,32 @@ is not specified")
set(command_echo "COMMAND_ECHO ${arg_COMMAND_ECHO}")
endif()
set(command_escaped "")
foreach(command_arg IN LISTS arg_COMMAND)
if(NOT command_arg MATCHES "^\\\${[^}]+}$" AND NOT command_arg MATCHES "^\".+\"$")
string(REPLACE "\"" "\\\"" command_arg "${command_arg}")
string(APPEND command_escaped " \"${command_arg}\"")
else()
# Assume that all arguments that passed as escaped variables can be empty when
# unwrapped during the script execution.
string(APPEND command_escaped " ${command_arg}")
endif()
endforeach()
file(GENERATE OUTPUT "${arg_OUTPUT_FILE}" CONTENT
"#!${CMAKE_COMMAND} -P
# Qt generated command wrapper
${environment_extras}
${pre_run}
execute_process(COMMAND ${extra_runner} ${arg_COMMAND}
execute_process(COMMAND ${extra_runner} ${command_escaped}
WORKING_DIRECTORY \"${arg_WORKING_DIRECTORY}\"
${command_echo}
RESULT_VARIABLE result
)
${post_run}
if(NOT result EQUAL 0)
string(JOIN \" \" full_command ${extra_runner} ${arg_COMMAND})
string(JOIN \" \" full_command ${extra_runner} ${command_escaped})
message(FATAL_ERROR \"\${full_command} execution failed with exit code \${result}.\")
endif()"
)

17
cmake/QtTestHelpers.cmake
View File

@ -948,7 +948,6 @@ for this function. Will be ignored")
if(arg_ARGS)
set(command_args ${arg_ARGS})# Avoid "${arg_ARGS}" usage and let cmake expand string to
# semicolon-separated list
qt_internal_wrap_command_arguments(command_args)
endif()
if(TARGET ${arg_COMMAND})
@ -974,13 +973,15 @@ for this function. Will be ignored")
get_target_property(crosscompiling_emulator ${executable_name} CROSSCOMPILING_EMULATOR)
if(NOT crosscompiling_emulator)
set(crosscompiling_emulator "")
else()
qt_internal_wrap_command_arguments(crosscompiling_emulator)
endif()
endif()
_qt_internal_create_command_script(COMMAND "${crosscompiling_emulator} \${env_test_runner} \
\"${executable_file}\" \${env_test_args} ${command_args}"
_qt_internal_create_command_script(COMMAND
${crosscompiling_emulator}
"\${env_test_runner}"
"${executable_file}"
"\${env_test_args}"
${command_args}
OUTPUT_FILE "${arg_OUTPUT_FILE}"
WORKING_DIRECTORY "${arg_WORKING_DIRECTORY}"
ENVIRONMENT ${arg_ENVIRONMENT}
@ -1044,12 +1045,6 @@ function(qt_internal_add_test_helper name)
endfunction()
function(qt_internal_wrap_command_arguments argument_list)
list(TRANSFORM ${argument_list} REPLACE "^(.+)$" "[=[\\1]=]")
list(JOIN ${argument_list} " " ${argument_list})
set(${argument_list} "${${argument_list}}" PARENT_SCOPE)
endfunction()
function(qt_internal_collect_command_environment out_path out_plugin_path)
# Get path to <qt_relocatable_install_prefix>/bin, as well as CMAKE_INSTALL_PREFIX/bin, and
# combine them with the PATH environment variable.