kolan
/
CMake
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Xcode: Honor -g0 to disable debugging (#12377) 

This commit changes ExtractFlag to remove all occurences
of a flag, and only save the last one. (i.e., the dominant
one according to GCC rules)
This commit is contained in:
Johan Bjork 2011-08-18 19:30:51 +02:00 committed by David Cole
parent 48ba9b2f19
commit cb22afc02c
1 changed files with 28 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
Source/cmGlobalXCodeGenerator.cxx
View File

@ -1219,20 +1219,31 @@ void cmGlobalXCodeGenerator::CreateCustomCommands(cmXCodeObject* buildPhases,
}
//----------------------------------------------------------------------------
// This function removes each occurence of the flag and returns the last one
// (i.e., 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)
{
if(pos == 0 || flags[pos-1]==' ')
{
while(pos < flags.size() && flags[pos] != ' ')
{
if(!saved)
{
retFlag += flags[pos];
}
flags[pos] = ' ';
pos++;
}
}
saved = true;
pos = flags.rfind(flag);
}
return retFlag;
}
@ -1870,7 +1881,17 @@ 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";
}