Browse Source

Merge topic 'ninja-dyndep-response-file'

4ef0eedb Ninja: always use response file for cmake_ninja_dyndep

Acked-by: Kitware Robot <[email protected]>
Merge-request: !1038
Brad King 8 years ago
parent
commit
37915a6930
1 changed files with 2 additions and 8 deletions
  1. 2 8
      Source/cmNinjaTargetGenerator.cxx

+ 2 - 8
Source/cmNinjaTargetGenerator.cxx

@@ -557,17 +557,11 @@ void cmNinjaTargetGenerator::WriteCompileRule(const std::string& lang)
     // Write the rule for ninja dyndep file generation.
     std::vector<std::string> ddCmds;
 
-#ifdef _WIN32
-    // Windows command line length is limited -> use response file for dyndep
-    // rules
+    // Command line length is almost always limited -> use response file for
+    // dyndep rules
     std::string ddRspFile = "$out.rsp";
     std::string ddRspContent = "$in";
     std::string ddInput = "@" + ddRspFile;
-#else
-    std::string ddRspFile;
-    std::string ddRspContent;
-    std::string ddInput = "$in";
-#endif
 
     // Run CMake dependency scanner on preprocessed output.
     std::string const cmake = this->GetLocalGenerator()->ConvertToOutputFormat(