diff options
| author | Peter Collingbourne <peter@pcc.me.uk> | 2012-03-26 03:16:15 (GMT) |
|---|---|---|
| committer | Peter Collingbourne <peter@pcc.me.uk> | 2012-03-26 03:46:12 (GMT) |
| commit | 6b5614f1642dcb75fd9de9571c813b21e4f9ce2b (patch) | |
| tree | a588742369a9c8dab2cff35b574121eed9669cff | |
| parent | c337c7e205259d5944b19c2a6d2007f490379ddc (diff) | |
| download | CMake-6b5614f1642dcb75fd9de9571c813b21e4f9ce2b.zip CMake-6b5614f1642dcb75fd9de9571c813b21e4f9ce2b.tar.gz CMake-6b5614f1642dcb75fd9de9571c813b21e4f9ce2b.tar.bz2 | |
Ninja: Substitute <OBJECT> and <CMAKE_C_COMPILER> in depfile flags
Patch by Amine Khaldi!
Also, start using the -MT flag to set a target name for depfiles.
This works around a bug observed in distcc, as explained in the
comment. Based on a patch by Alexander Usov.
| -rw-r--r-- | Modules/Compiler/GNU.cmake | 5 | ||||
| -rw-r--r-- | Source/cmNinjaTargetGenerator.cxx | 4 |
2 files changed, 8 insertions, 1 deletions
diff --git a/Modules/Compiler/GNU.cmake b/Modules/Compiler/GNU.cmake index bdcaf9d..c74c179 100644 --- a/Modules/Compiler/GNU.cmake +++ b/Modules/Compiler/GNU.cmake @@ -30,7 +30,10 @@ macro(__compiler_gnu lang) # in try_compile mode. GET_PROPERTY(_IN_TC GLOBAL PROPERTY IN_TRY_COMPILE) if(NOT _IN_TC OR CMAKE_FORCE_DEPFILES) - set(CMAKE_DEPFILE_FLAGS_${lang} "-MMD -MF <DEPFILE>") + # distcc does not transform -o to -MT when invoking the preprocessor + # internally, as it ought to. Work around this bug by setting -MT here + # even though it isn't strictly necessary. + set(CMAKE_DEPFILE_FLAGS_${lang} "-MMD -MT <OBJECT> -MF <DEPFILE>") endif() # Initial configuration flags. diff --git a/Source/cmNinjaTargetGenerator.cxx b/Source/cmNinjaTargetGenerator.cxx index 9acbc67..bea234f 100644 --- a/Source/cmNinjaTargetGenerator.cxx +++ b/Source/cmNinjaTargetGenerator.cxx @@ -331,6 +331,10 @@ cmNinjaTargetGenerator depfile = "$out.d"; cmSystemTools::ReplaceString(depfileFlagsStr, "<DEPFILE>", depfile.c_str()); + cmSystemTools::ReplaceString(depfileFlagsStr, "<OBJECT>", + "$out"); + cmSystemTools::ReplaceString(depfileFlagsStr, "<CMAKE_C_COMPILER>", + this->GetMakefile()->GetDefinition("CMAKE_C_COMPILER")); flags += " " + depfileFlagsStr; } vars.Flags = flags.c_str(); |