diff options
| author | Brad King <brad.king@kitware.com> | 2017-02-15 18:58:40 (GMT) |
|---|---|---|
| committer | Brad King <brad.king@kitware.com> | 2017-02-16 18:19:26 (GMT) |
| commit | ef399f9bc6b8e2ff917a5f2809346fb81fd954ae (patch) | |
| tree | 329a270ad32f04ebba493ea9abe5486fd74898a1 /Source/CTest/cmCTestUpdateHandler.cxx | |
| parent | bb4730dc5add8c5ac54b41efa5375189636429f3 (diff) | |
| download | CMake-ef399f9bc6b8e2ff917a5f2809346fb81fd954ae.zip CMake-ef399f9bc6b8e2ff917a5f2809346fb81fd954ae.tar.gz CMake-ef399f9bc6b8e2ff917a5f2809346fb81fd954ae.tar.bz2 | |
ctest_update: Refactor internal APIs to support more failure cases
Thread failure of VC tool commands through more APIs so that we can
detect when they fail. Defer updating of the individual VC tool usage
the future and just return true from them for now.
Diffstat (limited to 'Source/CTest/cmCTestUpdateHandler.cxx')
| -rw-r--r-- | Source/CTest/cmCTestUpdateHandler.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Source/CTest/cmCTestUpdateHandler.cxx b/Source/CTest/cmCTestUpdateHandler.cxx index 0998d59..2b5683a 100644 --- a/Source/CTest/cmCTestUpdateHandler.cxx +++ b/Source/CTest/cmCTestUpdateHandler.cxx @@ -198,7 +198,7 @@ int cmCTestUpdateHandler::ProcessHandler() xml.Element("UpdateType", cmCTestUpdateHandlerUpdateToString(this->UpdateType)); - vc->WriteXML(xml); + bool loadedMods = vc->WriteXML(xml); int localModifications = 0; int numUpdated = vc->GetPathCount(cmCTestVC::PathUpdated); @@ -246,7 +246,7 @@ int cmCTestUpdateHandler::ProcessHandler() xml.EndElement(); // UpdateReturnStatus xml.EndElement(); // Update xml.EndDocument(); - return updated ? numUpdated : -1; + return updated && loadedMods ? numUpdated : -1; } int cmCTestUpdateHandler::DetectVCS(const char* dir) |