From 665a30bc2dc78940ff3e1ec90d2f5bb7d329a71c Mon Sep 17 00:00:00 2001
From: Qingning Huo <qingninghuo@gmail.com>
Date: Fri, 20 Jan 2012 21:01:54 +0000
Subject: Mark CreateProcess "program not found" failure as non-fatal

---
 src/subprocess-win32.cc | 22 ++++++++++++++++++++--
 1 file changed, 20 insertions(+), 2 deletions(-)

diff --git a/src/subprocess-win32.cc b/src/subprocess-win32.cc
index cf61feb..e757f7e 100644
--- a/src/subprocess-win32.cc
+++ b/src/subprocess-win32.cc
@@ -95,7 +95,18 @@ bool Subprocess::Start(SubprocessSet* set, const string& command) {
                       /* inherit handles */ TRUE, 0,
                       NULL, NULL,
                       &startup_info, &process_info)) {
-    Win32Fatal("CreateProcess");
+    DWORD error = GetLastError();
+    if (error == ERROR_FILE_NOT_FOUND) { // file (program) not found error is treated as a normal build action failure
+      if (child_pipe)
+        CloseHandle(child_pipe);
+      CloseHandle(pipe_);
+      pipe_ = NULL;
+      // child_ is already NULL;
+      buf_ = "CreateProcess failed: The system cannot find the file specified.\n";
+      return true;
+    } else {
+      Win32Fatal("CreateProcess");    // pass all other errors to Win32Fatal
+    }
   }
 
   // Close pipe channel only used by the child.
@@ -139,6 +150,10 @@ void Subprocess::OnPipeReady() {
 }
 
 bool Subprocess::Finish() {
+  if (! child_) {
+    return false;
+  }
+
   // TODO: add error handling for all of these.
   WaitForSingleObject(child_, INFINITE);
 
@@ -170,7 +185,10 @@ SubprocessSet::~SubprocessSet() {
 }
 
 void SubprocessSet::Add(Subprocess* subprocess) {
-  running_.push_back(subprocess);
+  if (subprocess->child_)
+    running_.push_back(subprocess);
+  else
+    finished_.push(subprocess);
 }
 
 void SubprocessSet::DoWork() {
-- 
cgit v0.12


From 50c3475f47436d375d69e505e1c474c156e484fe Mon Sep 17 00:00:00 2001
From: Qingning Huo <qingninghuo@gmail.com>
Date: Wed, 25 Jan 2012 21:28:57 +0000
Subject: Add a test, NoSuchCommand.

---
 src/subprocess_test.cc | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/src/subprocess_test.cc b/src/subprocess_test.cc
index 47b7b56..840287b 100644
--- a/src/subprocess_test.cc
+++ b/src/subprocess_test.cc
@@ -45,6 +45,24 @@ TEST_F(SubprocessTest, BadCommandStderr) {
   EXPECT_NE("", subproc->GetOutput());
 }
 
+// Run a command that does not exist
+TEST_F(SubprocessTest, NoSuchCommand) {
+  Subprocess* subproc = new Subprocess;
+  EXPECT_TRUE(subproc->Start(&subprocs_, "ninja_no_such_command"));
+  subprocs_.Add(subproc);
+
+  while (!subproc->Done()) {
+    // Pretend we discovered that stderr was ready for writing.
+    subprocs_.DoWork();
+  }
+
+  EXPECT_FALSE(subproc->Finish());
+  EXPECT_NE("", subproc->GetOutput());
+#ifdef _WIN32
+  ASSERT_EQ("CreateProcess failed: The system cannot find the file specified.\n", subproc->GetOutput());
+#endif
+}
+
 TEST_F(SubprocessTest, SetWithSingle) {
   Subprocess* subproc = new Subprocess;
   EXPECT_TRUE(subproc->Start(&subprocs_, kSimpleCommand));
-- 
cgit v0.12