From 79859689005716ba38048d087ebe12f329e0229f Mon Sep 17 00:00:00 2001
From: Brad King <brad.king@kitware.com>
Date: Wed, 15 Jan 2014 16:33:22 -0500
Subject: [PATCH] Revert "Ninja: Avoid re-running CMake on next build after
 regeneration"

Revert commit 6fac24d7 (Ninja: Avoid re-running CMake on next build
after regeneration, 2014-01-09).  The files reported by the method
cmMakefile::GetOutputFiles() must cause CMake to re-run only if they are
missing and without considering a timestamp.  Therefore they cannot be
listed as outputs of the re-run rule or Ninja will re-run CMake every
time because the timestamp of configure_file outputs does not change.

Another solution will have to be found to the original problem.
---
 Source/cmGlobalNinjaGenerator.cxx | 29 +++++------------------------
 1 file changed, 5 insertions(+), 24 deletions(-)

diff --git a/Source/cmGlobalNinjaGenerator.cxx b/Source/cmGlobalNinjaGenerator.cxx
index 3e4e50635..61d02726e 100644
--- a/Source/cmGlobalNinjaGenerator.cxx
+++ b/Source/cmGlobalNinjaGenerator.cxx
@@ -1086,46 +1086,27 @@ void cmGlobalNinjaGenerator::WriteTargetRebuildManifest(std::ostream& os)
             /*depfile=*/ "",
             /*rspfile=*/ "",
             /*rspcontent*/ "",
-            /*restat=*/ true,
+            /*restat=*/ false,
             /*generator=*/ true);
 
-  cmNinjaDeps implicitDeps, outputs, temp;
-  outputs.push_back(NINJA_BUILD_FILE);
+  cmNinjaDeps implicitDeps;
   for (std::vector<cmLocalGenerator *>::const_iterator i =
        this->LocalGenerators.begin(); i != this->LocalGenerators.end(); ++i) {
     const std::vector<std::string>& lf = (*i)->GetMakefile()->GetListFiles();
     implicitDeps.insert(implicitDeps.end(), lf.begin(), lf.end());
 
     const std::vector<std::string>& of = (*i)->GetMakefile()->GetOutputFiles();
-    temp.insert(temp.end(), of.begin(), of.end());
+    implicitDeps.insert(implicitDeps.end(), of.begin(), of.end());
   }
-
-  //Add the CMakeCache.txt file to the implicit depends so that we catch
-  //when somebody manually modifies the file.
-  implicitDeps.push_back("CMakeCache.txt");
-
-  //make sure nothing is in implicit depends twice
   std::sort(implicitDeps.begin(), implicitDeps.end());
   implicitDeps.erase(std::unique(implicitDeps.begin(), implicitDeps.end()),
                      implicitDeps.end());
-
-
-  //make sure nothing is in outputs depends twice
-  std::sort(temp.begin(), temp.end());
-  temp.erase(std::unique(temp.begin(), temp.end()),
-                     temp.end());
-
-  //make sure that anything that is in implicitDeps is also NOT in outputs
-  std::set_difference(temp.begin(),
-                      temp.end(),
-                      implicitDeps.begin(),
-                      implicitDeps.end(),
-                      std::back_inserter(outputs));
+  implicitDeps.push_back("CMakeCache.txt");
 
   this->WriteBuild(os,
                    "Re-run CMake if any of its inputs changed.",
                    "RERUN_CMAKE",
-                   outputs,
+                   /*outputs=*/ cmNinjaDeps(1, NINJA_BUILD_FILE),
                    /*explicitDeps=*/ cmNinjaDeps(),
                    implicitDeps,
                    /*orderOnlyDeps=*/ cmNinjaDeps(),