ENH: Added cmTarget::GetLanguages method to centralize computation of the list of languages compiled in a target. Transformed NeedRequiresStep to use it.

This commit is contained in:
Brad King 2007-08-03 09:39:32 -04:00
parent 1c508b94ae
commit bf0c200f0a
2 changed files with 11 additions and 13 deletions

View File

@ -731,8 +731,7 @@ cmGlobalUnixMakefileGenerator3
makefileName = localName; makefileName = localName;
makefileName += "/build.make"; makefileName += "/build.make";
bool needRequiresStep = bool needRequiresStep = this->NeedRequiresStep(t->second);
this->NeedRequiresStep(lg,t->second.GetName());
lg->WriteDivider(ruleFileStream); lg->WriteDivider(ruleFileStream);
ruleFileStream ruleFileStream
@ -1168,18 +1167,17 @@ void cmGlobalUnixMakefileGenerator3::WriteHelpRule
bool cmGlobalUnixMakefileGenerator3 bool cmGlobalUnixMakefileGenerator3
::NeedRequiresStep(cmLocalUnixMakefileGenerator3 *lg,const char *name) ::NeedRequiresStep(cmTarget const& target)
{ {
std::map<cmStdString,cmLocalUnixMakefileGenerator3::IntegrityCheckSet>& std::set<cmStdString> languages;
checkSet = lg->GetIntegrityCheckSet()[name]; target.GetLanguages(languages);
for(std::map<cmStdString, for(std::set<cmStdString>::const_iterator l = languages.begin();
cmLocalUnixMakefileGenerator3::IntegrityCheckSet>::const_iterator l != languages.end(); ++l)
l = checkSet.begin(); l != checkSet.end(); ++l)
{ {
std::string name2 = "CMAKE_NEEDS_REQUIRES_STEP_"; std::string var = "CMAKE_NEEDS_REQUIRES_STEP_";
name2 += l->first; var += *l;
name2 += "_FLAG"; var += "_FLAG";
if(lg->GetMakefile()->GetDefinition(name2.c_str())) if(target.GetMakefile()->GetDefinition(var.c_str()))
{ {
return true; return true;
} }

View File

@ -165,7 +165,7 @@ protected:
cmTarget &target); cmTarget &target);
// does this generator need a requires step for any of its targets // does this generator need a requires step for any of its targets
bool NeedRequiresStep(cmLocalUnixMakefileGenerator3 *lg, const char *); bool NeedRequiresStep(cmTarget const&);
// Setup target names // Setup target names
virtual const char* GetAllTargetName() { return "all"; } virtual const char* GetAllTargetName() { return "all"; }