From 99cb10eda966592e5c369e3abe51b3442c31fd47 Mon Sep 17 00:00:00 2001 From: Aiden Grossman Date: Thu, 4 Sep 2025 21:02:35 +0000 Subject: [PATCH] [CI] Only invoke ninja for runtimes build if runtimes are specified Otherwise we end up running ninja without any targets specified which just builds the rest of the default enabled targets. This shouldn't have too much impact, but can involve building extra things that we don't need. This also makes the monolithic-windows.sh script consistent with the monolithic-windows.sh script. --- .ci/monolithic-windows.sh | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/.ci/monolithic-windows.sh b/.ci/monolithic-windows.sh index 14dea04c04291..f85d6e3d51b57 100755 --- a/.ci/monolithic-windows.sh +++ b/.ci/monolithic-windows.sh @@ -56,6 +56,8 @@ start-group "ninja" # Targets are not escaped as they are passed as separate arguments. ninja -C "${BUILD_DIR}" -k 0 ${targets} |& tee ninja.log -start-group "ninja runtimes" - -ninja -C "${BUILD_DIR}" -k 0 ${runtimes_targets} |& tee ninja_runtimes.log +if [[ "${runtime_targets}" != "" ]]; then + start-group "ninja runtimes" + + ninja -C "${BUILD_DIR}" -k 0 ${runtimes_targets} |& tee ninja_runtimes.log +fi