summaryrefslogtreecommitdiffstats
path: root/Source/cmNinjaTargetGenerator.cxx
diff options
context:
space:
mode:
authorBrad King <brad.king@kitware.com>2016-12-09 13:54:10 (GMT)
committerBrad King <brad.king@kitware.com>2016-12-09 13:59:17 (GMT)
commitb7771078ad1ade95a4ae310815be7971017e10e1 (patch)
treef6fa5fe9c79c121dc28dc903dc5a47fd0272c69c /Source/cmNinjaTargetGenerator.cxx
parentc58a5198e5eb8878ad111be2eafcfef75e94b54b (diff)
downloadCMake-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.cxx1
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);
}