Removing 'coverage combine' in test script

According to what was discovered in spack#6887, one of the problems is calling 'coverage combine' twice without the '-a' flag. This removes the first call within our test scripts.
epfl-scitas · Jan 16, 2018 · 22fb525 · 22fb525
1 parent f1a69a9
commit 22fb525
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 4 deletions.
diff --git a/share/spack/qa/run-build-tests b/share/spack/qa/run-build-tests
@@ -26,4 +26,3 @@ spack config get compilers
 
 # Run some build smoke tests, potentially with code coverage
 ${coverage_run} bin/spack install ${SPEC}
-${coverage_combine}
diff --git a/share/spack/qa/run-unit-tests b/share/spack/qa/run-unit-tests
@@ -29,4 +29,3 @@ ${coverage_run} bin/spack -p --lines 20 spec mpileaks
 
 # Run unit tests with code coverage
 ${coverage_run} bin/spack test "$@"
-${coverage_combine}
diff --git a/share/spack/qa/setup.sh b/share/spack/qa/setup.sh
@@ -14,11 +14,9 @@ SPACK_ROOT="$QA_DIR/../../.."
 if [[ "$COVERAGE" == true ]]; then
     coverage=coverage
     coverage_run="coverage run"
-    coverage_combine="coverage combine"
 else
     coverage=""
     coverage_run=""
-    coverage_combine=""
 fi
 
 #