From 32bac010ecb1b2d8ce7c9afe8fea0f3d65bd4780 Mon Sep 17 00:00:00 2001
From: Johan Bjork <phb@spotify.com>
Date: Thu, 18 Aug 2011 19:30:51 +0200
Subject: [PATCH] xcode: Honor -g0 to disable debugging.

Changes ExtractFlag to remove all occurances of a flag, and only save the last one.
(ie, the dominant one according to GCC rules)
---
 Source/cmGlobalXCodeGenerator.cxx |   32 +++++++++++++++++++++++++-------
 1 files changed, 25 insertions(+), 7 deletions(-)

diff --git a/Source/cmGlobalXCodeGenerator.cxx b/Source/cmGlobalXCodeGenerator.cxx
index 7396791..b6ef288 100644
--- a/Source/cmGlobalXCodeGenerator.cxx
+++ b/Source/cmGlobalXCodeGenerator.cxx
@@ -1197,19 +1197,28 @@ void cmGlobalXCodeGenerator::CreateCustomCommands(cmXCodeObject* buildPhases,
 }
 
 //----------------------------------------------------------------------------
+// This function removes each occurance of the flag and returns the last one
+// (ie, the dominant flag in GCC)
 std::string cmGlobalXCodeGenerator::ExtractFlag(const char* flag,
                                                 std::string& flags)
 {
   std::string retFlag;
-  std::string::size_type pos = flags.find(flag);
-  if(pos != flags.npos && (pos ==0 || flags[pos-1]==' '))
+  std::string::size_type pos = flags.rfind(flag);
+  bool saved = false;
+  while(pos != flags.npos)
     {
-    while(pos < flags.size() && flags[pos] != ' ')
+    if(pos == 0 || flags[pos-1]==' ')
       {
-      retFlag += flags[pos];
-      flags[pos] = ' ';
-      pos++;
+      while(pos < flags.size() && flags[pos] != ' ')
+        {
+        if(!saved)
+          retFlag += flags[pos];
+        flags[pos] = ' ';
+        pos++;
+        }
       }
+      saved = true;
+      pos = flags.rfind(flag);
     }
   return retFlag;
 }
@@ -1848,7 +1857,16 @@ void cmGlobalXCodeGenerator::CreateBuildSettings(cmTarget& target,
     flags += gflag;
     }
   const char* debugStr = "YES";
-  if(gflagc.size() ==0  && gflag.size() == 0)
+  //We can't set the XCode flag differently depending on the language, so put them back in this case.
+  if( (lang && strcmp(lang, "CXX") == 0) && gflag != gflagc)
+    {
+      cflags += " ";
+      cflags += gflagc;
+      flags += " ";
+      flags += gflag;
+      debugStr = "NO";
+    }
+  if( gflag == "-g0" || gflag.size() == 0)
     {
     debugStr = "NO";
     }
-- 
1.7.6