Xcode: Drop check for circular target dependencies
The modern cmComputeTargetDepends guarantees there will by no cycles. Drop the Xcode generator's check which only handled length 1 and 2 cycles anyway.
This commit is contained in:
parent
6535fcefeb
commit
781ea6d1ca
|
@ -2517,24 +2517,6 @@ std::string cmGlobalXCodeGenerator::GetOrCreateId(const char* name,
|
||||||
void cmGlobalXCodeGenerator::AddDependTarget(cmXCodeObject* target,
|
void cmGlobalXCodeGenerator::AddDependTarget(cmXCodeObject* target,
|
||||||
cmXCodeObject* dependTarget)
|
cmXCodeObject* dependTarget)
|
||||||
{
|
{
|
||||||
// make sure a target does not depend on itself
|
|
||||||
if(target == dependTarget)
|
|
||||||
{
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
// now avoid circular references if dependTarget already
|
|
||||||
// depends on target then skip it. Circular references crashes
|
|
||||||
// xcode
|
|
||||||
cmXCodeObject* dependTargetDepends =
|
|
||||||
dependTarget->GetObject("dependencies");
|
|
||||||
if(dependTargetDepends)
|
|
||||||
{
|
|
||||||
if(dependTargetDepends->HasObject(target->GetPBXTargetDependency()))
|
|
||||||
{
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
cmXCodeObject* targetdep = dependTarget->GetPBXTargetDependency();
|
cmXCodeObject* targetdep = dependTarget->GetPBXTargetDependency();
|
||||||
if(!targetdep)
|
if(!targetdep)
|
||||||
{
|
{
|
||||||
|
|
Loading…
Reference in New Issue