summaryrefslogtreecommitdiffstats
path: root/Source
diff options
context:
space:
mode:
authorPeter Kuemmel <syntheticpp@gmx.net>2012-02-19 18:21:05 (GMT)
committerPeter Kuemmel <syntheticpp@gmx.net>2012-02-19 18:21:05 (GMT)
commit21997cea4d8694077e8bcf9ba4cbd3d96a3f163c (patch)
treefbed02f39c3fe7974dc31b3bbb7c30632fdc70e7 /Source
parent3b847415154b13e401bf9f71042578ad21c9c1c9 (diff)
downloadCMake-21997cea4d8694077e8bcf9ba4cbd3d96a3f163c.zip
CMake-21997cea4d8694077e8bcf9ba4cbd3d96a3f163c.tar.gz
CMake-21997cea4d8694077e8bcf9ba4cbd3d96a3f163c.tar.bz2
Ninja: mark the Windows specific hacks with a comment only
Diffstat (limited to 'Source')
-rw-r--r--Source/cmNinjaNormalTargetGenerator.cxx3
-rw-r--r--Source/cmNinjaTargetGenerator.cxx5
2 files changed, 6 insertions, 2 deletions
diff --git a/Source/cmNinjaNormalTargetGenerator.cxx b/Source/cmNinjaNormalTargetGenerator.cxx
index 97b46bf..66714e9 100644
--- a/Source/cmNinjaNormalTargetGenerator.cxx
+++ b/Source/cmNinjaNormalTargetGenerator.cxx
@@ -172,7 +172,8 @@ cmNinjaNormalTargetGenerator
i != linkCmds.end();
++i)
{
-#ifdef ENABLE_WIN32_NINJA_HACKS
+#ifdef _WIN32
+ // TODO TARGET_IMPLIB is empty
cmSystemTools::ReplaceString(*i, "/implib:", "");
#endif
this->GetLocalGenerator()->ExpandRuleVariables(*i, vars);
diff --git a/Source/cmNinjaTargetGenerator.cxx b/Source/cmNinjaTargetGenerator.cxx
index cc6024b..1a8d13d 100644
--- a/Source/cmNinjaTargetGenerator.cxx
+++ b/Source/cmNinjaTargetGenerator.cxx
@@ -224,11 +224,14 @@ cmNinjaDeps cmNinjaTargetGenerator::ComputeLinkDeps() const
return cmNinjaDeps();
-#ifndef ENABLE_WIN32_NINJA_HACKS
+#ifndef _WIN32
const std::vector<std::string> &deps = cli->GetDepends();
cmNinjaDeps result(deps.size());
std::transform(deps.begin(), deps.end(), result.begin(), MapToNinjaPath());
#else
+ // TODO The ninja generator misses a lot on Windows.
+ // This hack avoids a problem when a Dll is build:
+ // It builds the .dll but uses the .lib which is then unknown to ninja.
cmNinjaDeps result;
for(unsigned i = 0; i < cli->GetItems().size(); ++i) {
if( cli->GetItems()[i].Target ) {