michaelh/zephyr
1
0
Fork 0
Code Releases Activity

tests/bsim/compile: Improve cmake_extra_args

Browse source 
Allow cmake_extra_args to contain multiple arguments

Signed-off-by: Alberto Escolar Piedras <alberto.escolar.piedras@nordicsemi.no>
This commit is contained in:
Alberto Escolar Piedras 2024-09-30 13:02:13 +02:00 • committed by Fabio Baltieri
commit 1460ef4419
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
tests/bsim/compile.source
View file

@ -25,7 +25,7 @@ function _compile(){
default_cmake_args=(-DCONFIG_COVERAGE=y -DCMAKE_EXPORT_COMPILE_COMMANDS=ON \
-DCONFIG_COMPILER_WARNINGS_AS_ERRORS=y -DCONFIG_ASSERT=y)
local cmake_args=(${cmake_args:-"${default_cmake_args[@]}"})
local cmake_extra_args="${cmake_extra_args:-""}"
local cmake_extra_args=(${cmake_extra_args:-""})
local ninja_args="${ninja_args:-""}"
local cc_flags="${cc_flags:-""}"
@ -61,7 +61,7 @@ function _compile(){
local cmake_cmd+=( -DOVERLAY_CONFIG="${conf_overlay}" \
-DEXTRA_CONF_FILE="${extra_conf_file}" \
${modules_arg} \
"${cmake_args[@]}" ${cc_flags:+-DCMAKE_C_FLAGS=${cc_flags}} ${cmake_extra_args})
"${cmake_args[@]}" ${cc_flags:+-DCMAKE_C_FLAGS=${cc_flags}} "${cmake_extra_args[@]}")
if [ -v sysbuild ]; then
local cmake_cmd+=( -DAPP_DIR=${app_root}/${app} ${ZEPHYR_BASE}/share/sysbuild/)
else