ENH: Avoid computing link information for static library targets. They do not link.

1 files changed, 25 insertions, 20 deletions

diff --git a/Source/cmGlobalXCodeGenerator.cxx b/Source/cmGlobalXCodeGenerator.cxx
index ed1af6473c..c2b0558498 100644
--- a/Source/cmGlobalXCodeGenerator.cxx
+++ b/Source/cmGlobalXCodeGenerator.cxx
@@ -2092,6 +2092,12 @@ void cmGlobalXCodeGenerator
       } 
     }
 
+  // Skip link information for static libraries.
+  if(cmtarget->GetType() == cmTarget::STATIC_LIBRARY)
+    {
+    return;
+    }
+
   // Loop over configuration types and set per-configuration info.
   for(std::vector<std::string>::iterator i =
         this->CurrentConfigurationTypes.begin();
@@ -2166,29 +2172,28 @@ void cmGlobalXCodeGenerator
     }
 
     // now add the link libraries
-    if(cmtarget->GetType() != cmTarget::STATIC_LIBRARY)
-      {
-      std::string linkLibs;
-      const char* sep = "";
-      typedef cmComputeLinkInformation::ItemVector ItemVector;
-      ItemVector const& libNames = cli.GetItems();
-      for(ItemVector::const_iterator li = libNames.begin();
-          li != libNames.end(); ++li)
+    {
+    std::string linkLibs;
+    const char* sep = "";
+    typedef cmComputeLinkInformation::ItemVector ItemVector;
+    ItemVector const& libNames = cli.GetItems();
+    for(ItemVector::const_iterator li = libNames.begin();
+        li != libNames.end(); ++li)
+      {
+      linkLibs += sep;
+      sep = " ";
+      if(li->IsPath)
         {
-        linkLibs += sep;
-        sep = " ";
-        if(li->IsPath)
-          {
-          linkLibs += this->XCodeEscapePath(li->Value.c_str());
-          }
-        else
-          {
-          linkLibs += li->Value;
-          }
+        linkLibs += this->XCodeEscapePath(li->Value.c_str());
+        }
+      else
+        {
+        linkLibs += li->Value;
         }
-      this->AppendBuildSettingAttribute(target, "OTHER_LDFLAGS",
-                                        linkLibs.c_str(), configName);
       }
+    this->AppendBuildSettingAttribute(target, "OTHER_LDFLAGS",
+                                      linkLibs.c_str(), configName);
+    }
     }
 }