Return non-zero status when errors occur.

Clean all was not returning non-zero when an error occur (like when
failing to remove a directory). This patch fix the problem and add a test
for it.

1 files changed, 4 insertions, 4 deletions

diff --git a/src/ninja_jumble.cc b/src/ninja_jumble.cc
index 1230546..537f48e 100644
--- a/src/ninja_jumble.cc
+++ b/src/ninja_jumble.cc
@@ -107,17 +107,17 @@ bool RealDiskInterface::MakeDir(const string& path) {
   return true;
 }
 
-bool RealDiskInterface::RemoveFile(const string& path) {
+int RealDiskInterface::RemoveFile(const string& path) {
   if (remove(path.c_str()) < 0) {
     switch (errno) {
       case ENOENT:
-        return false;
+        return 1;
       default:
         Error("remove(%s): %s", path.c_str(), strerror(errno));
-        return false;
+        return -1;
     }
   } else {
-    return true;
+    return 0;
   }
 }