diff options
author | Brad King <brad.king@kitware.com> | 2016-12-09 13:54:10 (GMT) |
---|---|---|
committer | Brad King <brad.king@kitware.com> | 2016-12-09 13:59:17 (GMT) |
commit | b7771078ad1ade95a4ae310815be7971017e10e1 (patch) | |
tree | f6fa5fe9c79c121dc28dc903dc5a47fd0272c69c /Source/cmNinjaTargetGenerator.cxx | |
parent | c58a5198e5eb8878ad111be2eafcfef75e94b54b (diff) | |
download | CMake-b7771078ad1ade95a4ae310815be7971017e10e1.zip CMake-b7771078ad1ade95a4ae310815be7971017e10e1.tar.gz CMake-b7771078ad1ade95a4ae310815be7971017e10e1.tar.bz2 |
Comment ExpandRuleVariables calls not preceded by launcher insertion
Call sites of `ExpandRuleVariables` are now responsible for inserting
the `RULE_LAUNCH_{COMPILE,LINK,CUSTOM}` values in rule command lines.
Audit all call sites and add a comment to each one that does not insert
a launcher to explain why.
Diffstat (limited to 'Source/cmNinjaTargetGenerator.cxx')
-rw-r--r-- | Source/cmNinjaTargetGenerator.cxx | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/Source/cmNinjaTargetGenerator.cxx b/Source/cmNinjaTargetGenerator.cxx index 1ac6cd4..489f120 100644 --- a/Source/cmNinjaTargetGenerator.cxx +++ b/Source/cmNinjaTargetGenerator.cxx @@ -1015,6 +1015,7 @@ void cmNinjaTargetGenerator::ExportObjectCompileCommand( for (std::vector<std::string>::iterator i = compileCmds.begin(); i != compileCmds.end(); ++i) { + // no launcher for CMAKE_EXPORT_COMPILE_COMMANDS rulePlaceholderExpander->ExpandRuleVariables(this->GetLocalGenerator(), *i, compileObjectVars); } |