Browse Source

Fixed a bug where it was possible for a test to be started twice if a lower-indexed test depended on it.

Zach Mullen 16 years ago
parent
commit
f7899c7445
1 changed files with 5 additions and 0 deletions
  1. 5 0
      Source/CTest/cmCTestMultiProcessHandler.cxx

+ 5 - 0
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)
         {