From f7899c7445121594d2e9324e4a3de815e1066255 Mon Sep 17 00:00:00 2001 From: Zach Mullen Date: Mon, 9 Nov 2009 16:07:47 -0500 Subject: [PATCH] Fixed a bug where it was possible for a test to be started twice if a lower-indexed test depended on it. --- Source/CTest/cmCTestMultiProcessHandler.cxx | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/Source/CTest/cmCTestMultiProcessHandler.cxx b/Source/CTest/cmCTestMultiProcessHandler.cxx index 4b3d4255b..cc238112a 100644 --- a/Source/CTest/cmCTestMultiProcessHandler.cxx +++ b/Source/CTest/cmCTestMultiProcessHandler.cxx @@ -196,6 +196,11 @@ void cmCTestMultiProcessHandler::StartNextTests() for(TestSet::iterator test = tests.begin(); test != tests.end(); ++test) { + //in case this test has already been started due to dependency + if(this->TestRunningMap[*test] || this->TestFinishMap[*test]) + { + continue; + } size_t processors = GetProcessorsUsed(*test); if(processors > numToStart) {