Xcode: Fix early termination on per-config source file error

In commit v3.1.0-rc1~687^2~4 (cmTarget: Make the source files depend on
the config, 2014-02-13) an early termination case was added to the Xcode
generator.  Fix handling of this case to actually abort all the
generation steps.  Otherwise some of the later steps are attempted
without the preconditions normally established by earlier steps,
possibly leading to a crash.
This commit is contained in:
Brad King 2015-01-19 08:27:15 -05:00
parent c118816d44
commit c0ff542c58
2 changed files with 29 additions and 13 deletions

View File

@ -966,7 +966,7 @@ struct cmSourceFilePathCompare
}; };
//---------------------------------------------------------------------------- //----------------------------------------------------------------------------
void bool
cmGlobalXCodeGenerator::CreateXCodeTargets(cmLocalGenerator* gen, cmGlobalXCodeGenerator::CreateXCodeTargets(cmLocalGenerator* gen,
std::vector<cmXCodeObject*>& std::vector<cmXCodeObject*>&
targets) targets)
@ -992,7 +992,12 @@ cmGlobalXCodeGenerator::CreateXCodeTargets(cmLocalGenerator* gen,
if(cmtarget.GetType() == cmTarget::UTILITY || if(cmtarget.GetType() == cmTarget::UTILITY ||
cmtarget.GetType() == cmTarget::GLOBAL_TARGET) cmtarget.GetType() == cmTarget::GLOBAL_TARGET)
{ {
targets.push_back(this->CreateUtilityTarget(cmtarget)); cmXCodeObject* t = this->CreateUtilityTarget(cmtarget);
if (!t)
{
return false;
}
targets.push_back(t);
continue; continue;
} }
@ -1000,7 +1005,7 @@ cmGlobalXCodeGenerator::CreateXCodeTargets(cmLocalGenerator* gen,
std::vector<cmSourceFile*> classes; std::vector<cmSourceFile*> classes;
if (!cmtarget.GetConfigCommonSourceFiles(classes)) if (!cmtarget.GetConfigCommonSourceFiles(classes))
{ {
return; return false;
} }
std::sort(classes.begin(), classes.end(), cmSourceFilePathCompare()); std::sort(classes.begin(), classes.end(), cmSourceFilePathCompare());
@ -1227,6 +1232,7 @@ cmGlobalXCodeGenerator::CreateXCodeTargets(cmLocalGenerator* gen,
targets.push_back(this->CreateXCodeTarget(cmtarget, buildPhases)); targets.push_back(this->CreateXCodeTarget(cmtarget, buildPhases));
} }
return true;
} }
//---------------------------------------------------------------------------- //----------------------------------------------------------------------------
@ -2940,7 +2946,7 @@ void cmGlobalXCodeGenerator
} }
//---------------------------------------------------------------------------- //----------------------------------------------------------------------------
void cmGlobalXCodeGenerator::CreateGroups(cmLocalGenerator* root, bool cmGlobalXCodeGenerator::CreateGroups(cmLocalGenerator* root,
std::vector<cmLocalGenerator*>& std::vector<cmLocalGenerator*>&
generators) generators)
{ {
@ -2983,7 +2989,7 @@ void cmGlobalXCodeGenerator::CreateGroups(cmLocalGenerator* root,
std::vector<cmSourceFile*> classes; std::vector<cmSourceFile*> classes;
if (!cmtarget.GetConfigCommonSourceFiles(classes)) if (!cmtarget.GetConfigCommonSourceFiles(classes))
{ {
return; return false;
} }
// Put cmSourceFile instances in proper groups: // Put cmSourceFile instances in proper groups:
for(std::vector<cmSourceFile*>::const_iterator s = classes.begin(); for(std::vector<cmSourceFile*>::const_iterator s = classes.begin();
@ -3016,6 +3022,7 @@ void cmGlobalXCodeGenerator::CreateGroups(cmLocalGenerator* root,
} }
} }
} }
return true;
} }
cmXCodeObject *cmGlobalXCodeGenerator cmXCodeObject *cmGlobalXCodeGenerator
@ -3136,7 +3143,7 @@ cmXCodeObject* cmGlobalXCodeGenerator
} }
//---------------------------------------------------------------------------- //----------------------------------------------------------------------------
void cmGlobalXCodeGenerator bool cmGlobalXCodeGenerator
::CreateXCodeObjects(cmLocalGenerator* root, ::CreateXCodeObjects(cmLocalGenerator* root,
std::vector<cmLocalGenerator*>& std::vector<cmLocalGenerator*>&
generators) generators)
@ -3217,7 +3224,10 @@ void cmGlobalXCodeGenerator
this->MainGroupChildren->AddObject(resourcesGroup); this->MainGroupChildren->AddObject(resourcesGroup);
// now create the cmake groups // now create the cmake groups
this->CreateGroups(root, generators); if (!this->CreateGroups(root, generators))
{
return false;
}
cmXCodeObject* productGroup = this->CreateObject(cmXCodeObject::PBXGroup); cmXCodeObject* productGroup = this->CreateObject(cmXCodeObject::PBXGroup);
productGroup->AddAttribute("name", this->CreateString("Products")); productGroup->AddAttribute("name", this->CreateString("Products"));
@ -3417,7 +3427,10 @@ void cmGlobalXCodeGenerator
{ {
if(!this->IsExcluded(root, *i)) if(!this->IsExcluded(root, *i))
{ {
this->CreateXCodeTargets(*i, targets); if (!this->CreateXCodeTargets(*i, targets))
{
return false;
}
} }
} }
// loop over all targets and add link and depend info // loop over all targets and add link and depend info
@ -3446,6 +3459,7 @@ void cmGlobalXCodeGenerator
} }
} }
this->RootObject->AddAttribute("targets", allTargets); this->RootObject->AddAttribute("targets", allTargets);
return true;
} }
//---------------------------------------------------------------------------- //----------------------------------------------------------------------------
@ -3632,8 +3646,10 @@ cmGlobalXCodeGenerator::OutputXCodeProject(cmLocalGenerator* root,
} }
} }
this->CreateXCodeObjects(root, if (!this->CreateXCodeObjects(root, generators))
generators); {
return;
}
std::string xcodeDir = root->GetMakefile()->GetStartOutputDirectory(); std::string xcodeDir = root->GetMakefile()->GetStartOutputDirectory();
xcodeDir += "/"; xcodeDir += "/";
xcodeDir += root->GetMakefile()->GetProjectName(); xcodeDir += root->GetMakefile()->GetProjectName();

View File

@ -91,7 +91,7 @@ private:
cmSourceGroup* sg); cmSourceGroup* sg);
cmXCodeObject* CreatePBXGroup(cmXCodeObject *parent, cmXCodeObject* CreatePBXGroup(cmXCodeObject *parent,
std::string name); std::string name);
void CreateGroups(cmLocalGenerator* root, bool CreateGroups(cmLocalGenerator* root,
std::vector<cmLocalGenerator*>& std::vector<cmLocalGenerator*>&
generators); generators);
std::string XCodeEscapePath(const char* p); std::string XCodeEscapePath(const char* p);
@ -151,7 +151,7 @@ private:
std::string ExtractFlag(const char* flag, std::string& flags); std::string ExtractFlag(const char* flag, std::string& flags);
// delete all objects in the this->XCodeObjects vector. // delete all objects in the this->XCodeObjects vector.
void ClearXCodeObjects(); void ClearXCodeObjects();
void CreateXCodeObjects(cmLocalGenerator* root, bool CreateXCodeObjects(cmLocalGenerator* root,
std::vector<cmLocalGenerator*>& generators); std::vector<cmLocalGenerator*>& generators);
void OutputXCodeProject(cmLocalGenerator* root, void OutputXCodeProject(cmLocalGenerator* root,
std::vector<cmLocalGenerator*>& generators); std::vector<cmLocalGenerator*>& generators);
@ -170,7 +170,7 @@ private:
cmXCodeObject* CreateXCodeSourceFile(cmLocalGenerator* gen, cmXCodeObject* CreateXCodeSourceFile(cmLocalGenerator* gen,
cmSourceFile* sf, cmSourceFile* sf,
cmTarget& cmtarget); cmTarget& cmtarget);
void CreateXCodeTargets(cmLocalGenerator* gen, bool CreateXCodeTargets(cmLocalGenerator* gen,
std::vector<cmXCodeObject*>&); std::vector<cmXCodeObject*>&);
bool IsHeaderFile(cmSourceFile*); bool IsHeaderFile(cmSourceFile*);
void AddDependTarget(cmXCodeObject* target, void AddDependTarget(cmXCodeObject* target,