From d8589e6437451ef174fc480d0dfc4e83e49b1b30 Mon Sep 17 00:00:00 2001 From: Ben Boeckel Date: Wed, 10 Dec 2014 17:22:21 -0500 Subject: [PATCH] ctest: count errors from scripts properly In the unlikely event that someone has a billion+ scripts (or some codepath returns negative numbers), we could overflow and make a pile of errors a non-error. This change also allows us to use flags for the error in the future rather than just "something went wrong". --- Source/CTest/cmCTestScriptHandler.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Source/CTest/cmCTestScriptHandler.cxx b/Source/CTest/cmCTestScriptHandler.cxx index 749eb5896..f958e7bba 100644 --- a/Source/CTest/cmCTestScriptHandler.cxx +++ b/Source/CTest/cmCTestScriptHandler.cxx @@ -183,7 +183,7 @@ int cmCTestScriptHandler::ProcessHandler() for (size_t i=0; i < this->ConfigurationScripts.size(); ++i) { // for each script run it - res += this->RunConfigurationScript + res |= this->RunConfigurationScript (cmSystemTools::CollapseFullPath(this->ConfigurationScripts[i]), this->ScriptProcessScope[i]); }