From c6cf2ca1087620b5b9b18115bdb991521a7742ff Mon Sep 17 00:00:00 2001
From: Ken Martin <ken.martin@kitware.com>
Date: Wed, 28 Mar 2007 10:58:00 -0400
Subject: [PATCH] BUG: fix checking of the return value for a build

---
 Source/CTest/cmCTestBuildHandler.cxx | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/Source/CTest/cmCTestBuildHandler.cxx b/Source/CTest/cmCTestBuildHandler.cxx
index bde76fe44..8f32ccd14 100644
--- a/Source/CTest/cmCTestBuildHandler.cxx
+++ b/Source/CTest/cmCTestBuildHandler.cxx
@@ -696,12 +696,13 @@ int cmCTestBuildHandler::RunMakeCommand(const char* command,
     cmCTestLog(this->CTest, HANDLER_VERBOSE_OUTPUT,
       "Command exited with the value: " << *retVal << std::endl);
     // if a non zero return value
-    if (retVal)
+    if (retVal && *retVal)
       {
       // If there was an error running command, report that on the dashboard.
       cmCTestBuildErrorWarning errorwarning;
       errorwarning.LogLine     = 1;
-      errorwarning.Text        = "*** WARNING non-zero return value from: ";
+      errorwarning.Text 
+        = "*** WARNING non-zero return value in ctest from: ";
       errorwarning.Text        += argv[0];
       errorwarning.PreContext  = "";
       errorwarning.PostContext = "";