Merge pull request RobotLocomotion#117 from rpoyner-tri/kcov-tool-fixes

coverage: Fix typos in kcov_tool integration
rpoyner-tri · Nov 10, 2020 · 425c823 · 425c823
2 parents 5c3003c + 346a2ca
commit 425c823
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/driver/configurations/bazel/step-build.cmake b/driver/configurations/bazel/step-build.cmake
@@ -164,8 +164,8 @@ endif()
 
 if(COVERAGE)
   set(KCOV_TOOL "${DASHBOARD_SOURCE_DIRECTORY}/tools/dynamic_analysis/kcov_tool")
-  if(EXISTS KCOV_TOOL)
-    execute_process(COMMAND KCOV_TOOL ci_merge)
+  if(EXISTS "${KCOV_TOOL}")
+    execute_process(COMMAND "${KCOV_TOOL}" ci_merge)
   endif()
   set(KCOV_MERGED "${DASHBOARD_SOURCE_DIRECTORY}/bazel-kcov/kcov-merged")
   execute_process(COMMAND "${CMAKE_COMMAND}" -E copy "${KCOV_MERGED}/cobertura.xml" "${KCOV_MERGED}/coverage.xml")