Skip to content

Commit 3f669d1

Browse files
committed
Don't quote DOTEST_EXTRA as they're part of a CMake list
Even though the dotest extra arguments can contain multiple flags, we shouldn't quote them because they're passed as a list to CMake. This also fixes the check above which was inspecting the wrong variable.
1 parent f08ec43 commit 3f669d1

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

utils/build-script-impl

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2495,8 +2495,8 @@ for host in "${ALL_HOSTS[@]}"; do
24952495
DOTEST_ARGS="--build-dir;${lldb_build_dir}/lldb-test-build.noindex;${LLDB_TEST_CATEGORIES};-t"
24962496

24972497
# Only set the extra arguments if they're not empty.
2498-
if [[ -z "${results_targets[@]}" ]]; then
2499-
DOTEST_ARGS="${DOTEST_ARGS};-E;\\\"${DOTEST_EXTRA}\\\""
2498+
if [[ -n "${DOTEST_EXTRA}" ]]; then
2499+
DOTEST_ARGS="${DOTEST_ARGS};-E;${DOTEST_EXTRA}"
25002500
fi
25012501

25022502
cmake_options=(

0 commit comments

Comments
 (0)