Skip to content

Commit 1460ef4

Browse files
aescolarfabiobaltieri
authored andcommitted
tests/bsim/compile: Improve cmake_extra_args
Allow cmake_extra_args to contain multiple arguments Signed-off-by: Alberto Escolar Piedras <[email protected]>
1 parent 9658b7d commit 1460ef4

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tests/bsim/compile.source

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ function _compile(){
2525
default_cmake_args=(-DCONFIG_COVERAGE=y -DCMAKE_EXPORT_COMPILE_COMMANDS=ON \
2626
-DCONFIG_COMPILER_WARNINGS_AS_ERRORS=y -DCONFIG_ASSERT=y)
2727
local cmake_args=(${cmake_args:-"${default_cmake_args[@]}"})
28-
local cmake_extra_args="${cmake_extra_args:-""}"
28+
local cmake_extra_args=(${cmake_extra_args:-""})
2929
local ninja_args="${ninja_args:-""}"
3030
local cc_flags="${cc_flags:-""}"
3131

@@ -61,7 +61,7 @@ function _compile(){
6161
local cmake_cmd+=( -DOVERLAY_CONFIG="${conf_overlay}" \
6262
-DEXTRA_CONF_FILE="${extra_conf_file}" \
6363
${modules_arg} \
64-
"${cmake_args[@]}" ${cc_flags:+-DCMAKE_C_FLAGS=${cc_flags}} ${cmake_extra_args})
64+
"${cmake_args[@]}" ${cc_flags:+-DCMAKE_C_FLAGS=${cc_flags}} "${cmake_extra_args[@]}")
6565
if [ -v sysbuild ]; then
6666
local cmake_cmd+=( -DAPP_DIR=${app_root}/${app} ${ZEPHYR_BASE}/share/sysbuild/)
6767
else

0 commit comments

Comments
 (0)