summaryrefslogtreecommitdiffstats
path: root/Source
diff options
context:
space:
mode:
authorBill Hoffman <bill.hoffman@kitware.com>2005-07-13 15:21:30 (GMT)
committerBill Hoffman <bill.hoffman@kitware.com>2005-07-13 15:21:30 (GMT)
commitaa47caab2db3961bfdd53036e5637df208b319d4 (patch)
tree1bd23995ec567a67fe8cccadd3750a215ea48829 /Source
parent1e72091e8631f1a94262a1a10b58d6b0a975db01 (diff)
downloadCMake-aa47caab2db3961bfdd53036e5637df208b319d4.zip
CMake-aa47caab2db3961bfdd53036e5637df208b319d4.tar.gz
CMake-aa47caab2db3961bfdd53036e5637df208b319d4.tar.bz2
FIX: apply patch from bug# 1965
Diffstat (limited to 'Source')
-rw-r--r--Source/cmLocalVisualStudio6Generator.cxx274
-rw-r--r--Source/cmLocalVisualStudio6Generator.h3
-rw-r--r--Source/cmLocalVisualStudio7Generator.cxx588
-rw-r--r--Source/cmLocalVisualStudio7Generator.h2
-rw-r--r--Source/cmMakefile.cxx385
-rw-r--r--Source/cmMakefile.h26
-rw-r--r--Source/cmSourceGroup.cxx85
-rw-r--r--Source/cmSourceGroup.h23
-rw-r--r--Source/cmSourceGroupCommand.cxx74
9 files changed, 886 insertions, 574 deletions
diff --git a/Source/cmLocalVisualStudio6Generator.cxx b/Source/cmLocalVisualStudio6Generator.cxx
index 8ac9e7b..c08cd66 100644
--- a/Source/cmLocalVisualStudio6Generator.cxx
+++ b/Source/cmLocalVisualStudio6Generator.cxx
@@ -207,8 +207,8 @@ void cmLocalVisualStudio6Generator::AddDSPBuildRule()
void cmLocalVisualStudio6Generator::WriteDSPFile(std::ostream& fout,
- const char *libName,
- cmTarget &target)
+ const char *libName,
+ cmTarget &target)
{
// if we should add regen rule then...
const char *suppRegenRule =
@@ -316,110 +316,127 @@ void cmLocalVisualStudio6Generator::WriteDSPFile(std::ostream& fout,
for(std::vector<cmSourceGroup>::const_iterator sg = sourceGroups.begin();
sg != sourceGroups.end(); ++sg)
{
- const std::vector<const cmSourceFile *> &sourceFiles =
- sg->GetSourceFiles();
- // If the group is empty, don't write it at all.
- if(sourceFiles.empty())
- {
- continue;
- }
+ this->WriteGroup(&(*sg), target, fout, libName);
+ }
+
+ // Write the DSP file's footer.
+ this->WriteDSPFooter(fout);
+}
+
+void cmLocalVisualStudio6Generator::WriteGroup(const cmSourceGroup *sg, cmTarget target, std::ostream &fout, const char *libName)
+{
+ const std::vector<const cmSourceFile *> &sourceFiles =
+ sg->GetSourceFiles();
+ // If the group is empty, don't write it at all.
+
+ if(sourceFiles.empty())
+ {
+ return;
+ }
+
+ // If the group has a name, write the header.
+ std::string name = sg->GetName();
+ if(name != "")
+ {
+ this->WriteDSPBeginGroup(fout, name.c_str(), "");
+ }
- // If the group has a name, write the header.
- std::string name = sg->GetName();
- if(name != "")
+ // Loop through each source in the source group.
+ for(std::vector<const cmSourceFile *>::const_iterator sf =
+ sourceFiles.begin(); sf != sourceFiles.end(); ++sf)
+ {
+ std::string source = (*sf)->GetFullPath();
+ const cmCustomCommand *command =
+ (*sf)->GetCustomCommand();
+ std::string compileFlags;
+ std::vector<std::string> depends;
+ const char* cflags = (*sf)->GetProperty("COMPILE_FLAGS");
+ if(cflags)
{
- this->WriteDSPBeginGroup(fout, name.c_str(), "");
+ compileFlags = cflags;
}
-
- // Loop through each source in the source group.
- for(std::vector<const cmSourceFile *>::const_iterator sf =
- sourceFiles.begin(); sf != sourceFiles.end(); ++sf)
+ const char* lang =
+ m_GlobalGenerator->GetLanguageFromExtension((*sf)->GetSourceExtension().c_str());
+ if(lang && strcmp(lang, "CXX") == 0)
{
- std::string source = (*sf)->GetFullPath();
- const cmCustomCommand *command =
- (*sf)->GetCustomCommand();
- std::string compileFlags;
- std::vector<std::string> depends;
- const char* cflags = (*sf)->GetProperty("COMPILE_FLAGS");
- if(cflags)
- {
- compileFlags = cflags;
- }
- const char* lang =
- m_GlobalGenerator->GetLanguageFromExtension((*sf)->GetSourceExtension().c_str());
- if(lang && strcmp(lang, "CXX") == 0)
+ // force a C++ file type
+ compileFlags += " /TP ";
+ }
+
+ // Check for extra object-file dependencies.
+ const char* dependsValue = (*sf)->GetProperty("OBJECT_DEPENDS");
+ if(dependsValue)
+ {
+ cmSystemTools::ExpandListArgument(dependsValue, depends);
+ }
+ if (source != libName || target.GetType() == cmTarget::UTILITY)
+ {
+ fout << "# Begin Source File\n\n";
+
+ // Tell MS-Dev what the source is. If the compiler knows how to
+ // build it, then it will.
+ fout << "SOURCE=" <<
+ this->ConvertToOptionallyRelativeOutputPath(source.c_str()) << "\n\n";
+ if(!depends.empty())
{
- // force a C++ file type
- compileFlags += " /TP ";
+ // Write out the dependencies for the rule.
+ fout << "USERDEP__HACK=";
+ for(std::vector<std::string>::const_iterator d = depends.begin();
+ d != depends.end(); ++d)
+ {
+ fout << "\\\n\t" <<
+ this->ConvertToOptionallyRelativeOutputPath(d->c_str());
+ }
+ fout << "\n";
}
-
- // Check for extra object-file dependencies.
- const char* dependsValue = (*sf)->GetProperty("OBJECT_DEPENDS");
- if(dependsValue)
+ if (command)
{
- cmSystemTools::ExpandListArgument(dependsValue, depends);
+ std::string script =
+ this->ConstructScript(command->GetCommandLines(), "\\\n\t");
+ const char* comment = command->GetComment();
+ const char* flags = compileFlags.size() ? compileFlags.c_str(): 0;
+ this->WriteCustomRule(fout, source.c_str(), script.c_str(),
+ (*comment?comment:"Custom Rule"),
+ command->GetDepends(),
+ command->GetOutput(), flags);
}
- if (source != libName || target.GetType() == cmTarget::UTILITY)
+ else if(compileFlags.size())
{
- fout << "# Begin Source File\n\n";
-
- // Tell MS-Dev what the source is. If the compiler knows how to
- // build it, then it will.
- fout << "SOURCE=" <<
- this->ConvertToOptionallyRelativeOutputPath(source.c_str()) << "\n\n";
- if(!depends.empty())
- {
- // Write out the dependencies for the rule.
- fout << "USERDEP__HACK=";
- for(std::vector<std::string>::const_iterator d = depends.begin();
- d != depends.end(); ++d)
- {
- fout << "\\\n\t" <<
- this->ConvertToOptionallyRelativeOutputPath(d->c_str());
+ for(std::vector<std::string>::iterator i
+ = m_Configurations.begin(); i != m_Configurations.end(); ++i)
+ {
+ if (i == m_Configurations.begin())
+ {
+ fout << "!IF \"$(CFG)\" == " << i->c_str() << std::endl;
}
- fout << "\n";
- }
- if (command)
- {
- std::string script =
- this->ConstructScript(command->GetCommandLines(), "\\\n\t");
- const char* comment = command->GetComment();
- const char* flags = compileFlags.size() ? compileFlags.c_str(): 0;
- this->WriteCustomRule(fout, source.c_str(), script.c_str(),
- (*comment?comment:"Custom Rule"),
- command->GetDepends(),
- command->GetOutput(), flags);
- }
- else if(compileFlags.size())
- {
- for(std::vector<std::string>::iterator i
- = m_Configurations.begin(); i != m_Configurations.end(); ++i)
- {
- if (i == m_Configurations.begin())
- {
- fout << "!IF \"$(CFG)\" == " << i->c_str() << std::endl;
- }
- else
- {
- fout << "!ELSEIF \"$(CFG)\" == " << i->c_str() << std::endl;
- }
- fout << "\n# ADD CPP " << compileFlags << "\n\n";
- }
- fout << "!ENDIF\n\n";
- }
- fout << "# End Source File\n";
+ else
+ {
+ fout << "!ELSEIF \"$(CFG)\" == " << i->c_str() << std::endl;
+ }
+ fout << "\n# ADD CPP " << compileFlags << "\n\n";
+ }
+ fout << "!ENDIF\n\n";
}
+ fout << "# End Source File\n";
}
+ }
+
+ std::vector<cmSourceGroup> children = sg->GetGroupChildren();
+
+ for(unsigned int i=0;i<children.size();++i)
+ {
+ this->WriteGroup(&children[i], target, fout, libName);
+ }
+
+
+
- // If the group has a name, write the footer.
- if(name != "")
- {
- this->WriteDSPEndGroup(fout);
- }
- }
+ // If the group has a name, write the footer.
+ if(name != "")
+ {
+ this->WriteDSPEndGroup(fout);
+ }
- // Write the DSP file's footer.
- this->WriteDSPFooter(fout);
}
@@ -458,13 +475,13 @@ cmLocalVisualStudio6Generator
}
void cmLocalVisualStudio6Generator::WriteCustomRule(std::ostream& fout,
- const char* source,
- const char* command,
- const char* comment,
- const std::vector<std::string>& depends,
- const char *output,
- const char* flags
- )
+ const char* source,
+ const char* command,
+ const char* comment,
+ const std::vector<std::string>& depends,
+ const char *output,
+ const char* flags
+ )
{
std::vector<std::string>::iterator i;
for(i = m_Configurations.begin(); i != m_Configurations.end(); ++i)
@@ -544,8 +561,8 @@ void cmLocalVisualStudio6Generator::WriteCustomRule(std::ostream& fout,
void cmLocalVisualStudio6Generator::WriteDSPBeginGroup(std::ostream& fout,
- const char* group,
- const char* filter)
+ const char* group,
+ const char* filter)
{
fout << "# Begin Group \"" << group << "\"\n"
"# PROP Default_Filter \"" << filter << "\"\n";
@@ -763,9 +780,11 @@ void cmLocalVisualStudio6Generator
libPath = m_Makefile->GetDefinition("LIBRARY_OUTPUT_PATH");
}
std::string exePath = "";
+ std::string exePathDebug = "";
if (m_Makefile->GetDefinition("EXECUTABLE_OUTPUT_PATH"))
{
exePath = m_Makefile->GetDefinition("EXECUTABLE_OUTPUT_PATH");
+
}
if(libPath.size())
{
@@ -952,13 +971,37 @@ void cmLocalVisualStudio6Generator
extraLinkOptions =
m_Makefile->GetRequiredDefinition("CMAKE_EXE_LINKER_FLAGS");
+
// if the executable has an output name then add the appropriate flag
if (target.GetProperty("OUTPUT_NAME"))
{
+ std::string outputname = target.GetProperty("OUTPUT_NAME");
libMultiLineOptions += "# ADD LINK32 /out:";
- libMultiLineOptions += target.GetProperty("OUTPUT_NAME");
+ libMultiLineOptions += outputname;
libMultiLineOptions += " \n";
}
+ else
+ {
+
+ libMultiLineOptions += "# ADD LINK32 /out:\"";
+
+ if(exePath != "")
+ libMultiLineOptions += exePath + "/" + libName + ".exe";
+ else
+ libMultiLineOptions += std::string(libName) + ".exe";
+
+ libMultiLineOptions += "\"\n";
+
+
+ libMultiLineOptionsForDebug += "# ADD LINK32 /out:\"";
+
+ if(exePath != "")
+ libMultiLineOptionsForDebug += exePath + "/" + libName + "D.exe";
+ else
+ libMultiLineOptionsForDebug += std::string(libName) + "D.exe";
+
+ libMultiLineOptionsForDebug += "\"\n";
+ }
}
if(target.GetType() == cmTarget::SHARED_LIBRARY)
{
@@ -1087,16 +1130,35 @@ void cmLocalVisualStudio6Generator
cmSystemTools::ReplaceString(line, "LIBRARY_OUTPUT_PATH",
removeQuotes(
this->ConvertToOptionallyRelativeOutputPath(libPath.c_str())).c_str());
- cmSystemTools::ReplaceString(line, "EXECUTABLE_OUTPUT_PATH",
- removeQuotes(
- this->ConvertToOptionallyRelativeOutputPath(exePath.c_str())).c_str());
+
+
+ if (!m_Makefile->GetDefinition("EXECUTABLE_OUTPUT_PATH_OVERRIDE") || exePath == "")
+ {
+ cmSystemTools::ReplaceString(line, "EXECUTABLE_OUTPUT_PATH",
+ removeQuotes(
+ this->ConvertToOptionallyRelativeOutputPath(exePath.c_str())).c_str());
+ } else
+ {
+ cmSystemTools::ReplaceString(line, "EXECUTABLE_OUTPUT_PATHRelease",
+ removeQuotes(
+ this->ConvertToOptionallyRelativeOutputPath(exePath.c_str())).c_str());
+ cmSystemTools::ReplaceString(line, "EXECUTABLE_OUTPUT_PATHDebug",
+ removeQuotes(
+ this->ConvertToOptionallyRelativeOutputPath(exePath.c_str())).c_str());
+ cmSystemTools::ReplaceString(line, "EXECUTABLE_OUTPUT_PATHMinSizeRel",
+ removeQuotes(
+ this->ConvertToOptionallyRelativeOutputPath(exePath.c_str())).c_str());
+ cmSystemTools::ReplaceString(line, "EXECUTABLE_OUTPUT_PATHRelWithDebInfo",
+ removeQuotes(
+ this->ConvertToOptionallyRelativeOutputPath(exePath.c_str())).c_str());
+ }
cmSystemTools::ReplaceString(line,
"EXTRA_DEFINES",
m_Makefile->GetDefineFlags());
const char* debugPostfix
= m_Makefile->GetDefinition("CMAKE_DEBUG_POSTFIX");
cmSystemTools::ReplaceString(line, "DEBUG_POSTFIX",
- debugPostfix?debugPostfix:"");
+ debugPostfix?debugPostfix:"");
// store flags for each configuration
std::string flags = " ";
std::string flagsRelease = " ";
diff --git a/Source/cmLocalVisualStudio6Generator.h b/Source/cmLocalVisualStudio6Generator.h
index bf09f5c..035b516 100644
--- a/Source/cmLocalVisualStudio6Generator.h
+++ b/Source/cmLocalVisualStudio6Generator.h
@@ -90,7 +90,8 @@ private:
void AddUtilityCommandHack(cmTarget& target, int count,
std::vector<std::string>& depends,
const cmCustomCommandLines& commandLines);
-
+ void WriteGroup(const cmSourceGroup *sg, cmTarget target,
+ std::ostream &fout, const char *libName);
std::string CreateTargetRules(cmTarget &target,
const char *libName);
std::string m_IncludeOptions;
diff --git a/Source/cmLocalVisualStudio7Generator.cxx b/Source/cmLocalVisualStudio7Generator.cxx
index d081d20..04a36f1 100644
--- a/Source/cmLocalVisualStudio7Generator.cxx
+++ b/Source/cmLocalVisualStudio7Generator.cxx
@@ -285,9 +285,9 @@ cmVS7FlagTable cmLocalVisualStudio7GeneratorLinkFlagTable[] =
void cmLocalVisualStudio7Generator::WriteConfiguration(std::ostream& fout,
- const char* configName,
- const char *libName,
- cmTarget &target)
+ const char* configName,
+ const char *libName,
+ cmTarget &target)
{
// create a map of xml tags to the values they should have in the output
// for example, "BufferSecurityCheck" = "TRUE"
@@ -386,7 +386,7 @@ void cmLocalVisualStudio7Generator::WriteConfiguration(std::ostream& fout,
flags += flagsDebug;
pre = "WIN32,_DEBUG,_WINDOWS";
std::string libpath = m_LibraryOutputPath +
- "$(OutDir)/" + libName + debugPostfix + ".pdb";
+ "$(OutDir)/" + libName + debugPostfix + ".pdb";
programDatabase = "\t\t\t\tProgramDatabaseFileName=\"";
programDatabase += libpath;
programDatabase += "\"";
@@ -416,7 +416,7 @@ void cmLocalVisualStudio7Generator::WriteConfiguration(std::ostream& fout,
pre = "WIN32,_WINDOWS";
flags += flagsDebugRel;
std::string libpath = m_LibraryOutputPath +
- "$(OutDir)/" + libName + debugPostfix + ".pdb";
+ "$(OutDir)/" + libName + debugPostfix + ".pdb";
programDatabase = "\t\t\t\tProgramDatabaseFileName=\"";
programDatabase += libpath;
programDatabase += "\"";
@@ -508,7 +508,7 @@ void cmLocalVisualStudio7Generator::WriteConfiguration(std::ostream& fout,
fout << "\t\t\t\tObjectFile=\"" << configName << "\\\"\n";
if(m_Makefile->IsOn("CMAKE_VERBOSE_MAKEFILE"))
{
- fout << "\t\t\t\tSuppressStartupBanner=\"FALSE\"\n";
+ fout << "\t\t\t\tSuppressStartupBanner=\"FALSE\"\n";
}
if(programDatabase.size())
@@ -574,9 +574,9 @@ void cmLocalVisualStudio7Generator::FillFlagMapFromCommandFlags(
void cmLocalVisualStudio7Generator::OutputBuildTool(std::ostream& fout,
- const char* configName,
- const char *libName,
- cmTarget &target)
+ const char* configName,
+ const char *libName,
+ cmTarget &target)
{
std::string temp;
std::string debugPostfix = "";
@@ -616,185 +616,191 @@ void cmLocalVisualStudio7Generator::OutputBuildTool(std::ostream& fout,
{
case cmTarget::STATIC_LIBRARY:
{
- std::string libpath = m_LibraryOutputPath +
- "$(OutDir)/" + libName + debugPostfix + ".lib";
- fout << "\t\t\t<Tool\n"
- << "\t\t\t\tName=\"VCLibrarianTool\"\n";
- if(const char* libflags = target.GetProperty("STATIC_LIBRARY_FLAGS"))
- {
- fout << "\t\t\t\tAdditionalOptions=\"" << libflags << "\"\n";
- }
- fout << "\t\t\t\tOutputFile=\""
- << this->ConvertToXMLOutputPathSingle(libpath.c_str()) << ".\"/>\n";
- break;
+ std::string libpath = m_LibraryOutputPath +
+ "$(OutDir)/" + libName + debugPostfix + ".lib";
+ fout << "\t\t\t<Tool\n"
+ << "\t\t\t\tName=\"VCLibrarianTool\"\n";
+ if(const char* libflags = target.GetProperty("STATIC_LIBRARY_FLAGS"))
+ {
+ fout << "\t\t\t\tAdditionalOptions=\"" << libflags << "\"\n";
+ }
+ fout << "\t\t\t\tOutputFile=\""
+ << this->ConvertToXMLOutputPathSingle(libpath.c_str()) << ".\"/>\n";
+ break;
}
case cmTarget::SHARED_LIBRARY:
case cmTarget::MODULE_LIBRARY:
+ {
+ fout << "\t\t\t<Tool\n"
+ << "\t\t\t\tName=\"VCLinkerTool\"\n"
+ << "\t\t\t\tAdditionalOptions=\"/MACHINE:I386";
+ if(extraLinkOptions.size())
{
- fout << "\t\t\t<Tool\n"
- << "\t\t\t\tName=\"VCLinkerTool\"\n"
- << "\t\t\t\tAdditionalOptions=\"/MACHINE:I386";
- if(extraLinkOptions.size())
- {
- fout << " " << cmLocalVisualStudio7Generator::EscapeForXML(
- extraLinkOptions.c_str()).c_str();
- }
- fout << "\"\n"
- << "\t\t\t\tAdditionalDependencies=\" odbc32.lib odbccp32.lib ";
- this->OutputLibraries(fout, configName, libName, target);
- fout << "\"\n";
- temp = m_LibraryOutputPath;
- temp += configName;
- temp += "/";
- temp += libName;
- temp += debugPostfix;
- temp += ".dll";
- fout << "\t\t\t\tOutputFile=\""
- << this->ConvertToXMLOutputPathSingle(temp.c_str()) << "\"\n";
- for(std::map<cmStdString, cmStdString>::iterator i = flagMap.begin();
- i != flagMap.end(); ++i)
- {
- fout << "\t\t\t\t" << i->first << "=\"" << i->second << "\"\n";
- }
+ fout << " " << cmLocalVisualStudio7Generator::EscapeForXML(
+ extraLinkOptions.c_str()).c_str();
+ }
+ fout << "\"\n"
+ << "\t\t\t\tAdditionalDependencies=\" odbc32.lib odbccp32.lib ";
+ this->OutputLibraries(fout, configName, libName, target);
+ fout << "\"\n";
+ temp = m_LibraryOutputPath;
+ temp += configName;
+ temp += "/";
+ temp += libName;
+ temp += debugPostfix;
+ temp += ".dll";
+ fout << "\t\t\t\tOutputFile=\""
+ << this->ConvertToXMLOutputPathSingle(temp.c_str()) << "\"\n";
+ for(std::map<cmStdString, cmStdString>::iterator i = flagMap.begin();
+ i != flagMap.end(); ++i)
+ {
+ fout << "\t\t\t\t" << i->first << "=\"" << i->second << "\"\n";
+ }
- if(m_Makefile->IsOn("CMAKE_VERBOSE_MAKEFILE"))
- {
- fout << "\t\t\t\tSuppressStartupBanner=\"FALSE\"\n";
- }
- else
- {
- fout << "\t\t\t\tSuppressStartupBanner=\"TRUE\"\n";
- }
+ if(m_Makefile->IsOn("CMAKE_VERBOSE_MAKEFILE"))
+ {
+ fout << "\t\t\t\tSuppressStartupBanner=\"FALSE\"\n";
+ }
+ else
+ {
+ fout << "\t\t\t\tSuppressStartupBanner=\"TRUE\"\n";
+ }
- fout << "\t\t\t\tAdditionalLibraryDirectories=\"";
- this->OutputLibraryDirectories(fout, configName, libName, target);
- fout << "\"\n";
- this->OutputModuleDefinitionFile(fout, target);
- temp = m_LibraryOutputPath;
- temp += "$(OutDir)/";
- temp += libName;
- temp += debugPostfix;
- temp += ".pdb";
- fout << "\t\t\t\tProgramDatabaseFile=\"" <<
- this->ConvertToXMLOutputPathSingle(temp.c_str()) << "\"\n";
- if(strcmp(configName, "Debug") == 0
- || strcmp(configName, "RelWithDebInfo") == 0)
- {
- fout << "\t\t\t\tGenerateDebugInformation=\"TRUE\"\n";
- }
- const char* linkLanguage = target.GetLinkerLanguage(this->GetGlobalGenerator());
- if(!linkLanguage)
- {
- cmSystemTools::Error("CMake can not determine linker language for target:",
- target.GetName());
- return;
- }
- std::string stackVar = "CMAKE_";
- stackVar += linkLanguage;
- stackVar += "_STACK_SIZE";
- const char* stackVal = m_Makefile->GetDefinition(stackVar.c_str());
- if(stackVal)
- {
- fout << "\t\t\t\tStackReserveSize=\"" << stackVal << "\"\n";
- }
- temp = m_LibraryOutputPath;
- temp += configName;
- temp += "/";
- temp += libName;
- temp += debugPostfix;
- temp += ".lib";
- fout << "\t\t\t\tImportLibrary=\"" << this->ConvertToXMLOutputPathSingle(temp.c_str()) << "\"/>\n";
+ fout << "\t\t\t\tAdditionalLibraryDirectories=\"";
+ this->OutputLibraryDirectories(fout, configName, libName, target);
+ fout << "\"\n";
+ this->OutputModuleDefinitionFile(fout, target);
+ temp = m_LibraryOutputPath;
+ temp += "$(OutDir)/";
+ temp += libName;
+ temp += debugPostfix;
+ temp += ".pdb";
+ fout << "\t\t\t\tProgramDatabaseFile=\"" <<
+ this->ConvertToXMLOutputPathSingle(temp.c_str()) << "\"\n";
+ if(strcmp(configName, "Debug") == 0
+ || strcmp(configName, "RelWithDebInfo") == 0)
+ {
+ fout << "\t\t\t\tGenerateDebugInformation=\"TRUE\"\n";
}
- break;
- case cmTarget::EXECUTABLE:
+ const char* linkLanguage = target.GetLinkerLanguage(this->GetGlobalGenerator());
+ if(!linkLanguage)
+ {
+ cmSystemTools::Error("CMake can not determine linker language for target:",
+ target.GetName());
+ return;
+ }
+ std::string stackVar = "CMAKE_";
+ stackVar += linkLanguage;
+ stackVar += "_STACK_SIZE";
+ const char* stackVal = m_Makefile->GetDefinition(stackVar.c_str());
+ if(stackVal)
{
+ fout << "\t\t\t\tStackReserveSize=\"" << stackVal << "\"\n";
+ }
+ temp = m_LibraryOutputPath;
+ temp += configName;
+ temp += "/";
+ temp += libName;
+ temp += debugPostfix;
+ temp += ".lib";
+ fout << "\t\t\t\tImportLibrary=\"" << this->ConvertToXMLOutputPathSingle(temp.c_str()) << "\"/>\n";
+ }
+ break;
+ case cmTarget::EXECUTABLE:
+ {
- fout << "\t\t\t<Tool\n"
- << "\t\t\t\tName=\"VCLinkerTool\"\n"
- << "\t\t\t\tAdditionalOptions=\"/MACHINE:I386";
- if(extraLinkOptions.size())
- {
- fout << " " << cmLocalVisualStudio7Generator::EscapeForXML(
- extraLinkOptions.c_str()).c_str();
- }
- fout << "\"\n"
- << "\t\t\t\tAdditionalDependencies=\""
- << m_Makefile->GetRequiredDefinition("CMAKE_STANDARD_LIBRARIES")
- << " ";
- this->OutputLibraries(fout, configName, libName, target);
- fout << "\"\n";
- temp = m_ExecutableOutputPath;
+ fout << "\t\t\t<Tool\n"
+ << "\t\t\t\tName=\"VCLinkerTool\"\n"
+ << "\t\t\t\tAdditionalOptions=\"/MACHINE:I386";
+ if(extraLinkOptions.size())
+ {
+ fout << " " << cmLocalVisualStudio7Generator::EscapeForXML(
+ extraLinkOptions.c_str()).c_str();
+ }
+ fout << "\"\n"
+ << "\t\t\t\tAdditionalDependencies=\""
+ << m_Makefile->GetRequiredDefinition("CMAKE_STANDARD_LIBRARIES")
+ << " ";
+ this->OutputLibraries(fout, configName, libName, target);
+ fout << "\"\n";
+ temp = m_ExecutableOutputPath;
+
+ if (!m_Makefile->GetDefinition("EXECUTABLE_OUTPUT_PATH_OVERRIDE"))
temp += configName;
- temp += "/";
- // do we have a different executable name?
- if (target.GetProperty("OUTPUT_NAME"))
- {
- temp += target.GetProperty("OUTPUT_NAME");
- }
- else
- {
- temp += libName;
- }
- temp += ".exe";
- fout << "\t\t\t\tOutputFile=\"" << this->ConvertToXMLOutputPathSingle(temp.c_str()) << "\"\n";
- for(std::map<cmStdString, cmStdString>::iterator i = flagMap.begin();
- i != flagMap.end(); ++i)
- {
- fout << "\t\t\t\t" << i->first << "=\"" << i->second << "\"\n";
- }
- if(m_Makefile->IsOn("CMAKE_VERBOSE_MAKEFILE"))
- {
- fout << "\t\t\t\tSuppressStartupBanner=\"FALSE\"\n";
- }
- else
- {
- fout << "\t\t\t\tSuppressStartupBanner=\"TRUE\"\n";
- }
+ temp += "/";
+
+ // do we have a different executable name?
+ if (target.GetProperty("OUTPUT_NAME"))
+ {
+ temp += target.GetProperty("OUTPUT_NAME");
+ }
+ else
+ {
+ temp += libName;
+ }
+
+ temp += debugPostfix;
+
+ temp += ".exe";
+ fout << "\t\t\t\tOutputFile=\"" << this->ConvertToXMLOutputPathSingle(temp.c_str()) << "\"\n";
+ for(std::map<cmStdString, cmStdString>::iterator i = flagMap.begin();
+ i != flagMap.end(); ++i)
+ {
+ fout << "\t\t\t\t" << i->first << "=\"" << i->second << "\"\n";
+ }
+ if(m_Makefile->IsOn("CMAKE_VERBOSE_MAKEFILE"))
+ {
+ fout << "\t\t\t\tSuppressStartupBanner=\"FALSE\"\n";
+ }
+ else
+ {
+ fout << "\t\t\t\tSuppressStartupBanner=\"TRUE\"\n";
+ }
- fout << "\t\t\t\tAdditionalLibraryDirectories=\"";
- this->OutputLibraryDirectories(fout, configName, libName, target);
- fout << "\"\n";
- fout << "\t\t\t\tProgramDatabaseFile=\"" << m_LibraryOutputPath
- << "$(OutDir)\\" << libName << ".pdb\"\n";
- if(strcmp(configName, "Debug") == 0
- || strcmp(configName, "RelWithDebInfo") == 0)
- {
- fout << "\t\t\t\tGenerateDebugInformation=\"TRUE\"\n";
- }
- if ( target.GetPropertyAsBool("WIN32_EXECUTABLE") )
- {
- fout << "\t\t\t\tSubSystem=\"2\"\n";
- }
- else
- {
- fout << "\t\t\t\tSubSystem=\"1\"\n";
- }
- const char* linkLanguage = target.GetLinkerLanguage(this->GetGlobalGenerator());
- if(!linkLanguage)
- {
- cmSystemTools::Error("CMake can not determine linker language for target:",
- target.GetName());
- return;
- }
- std::string stackVar = "CMAKE_";
- stackVar += linkLanguage;
- stackVar += "_STACK_SIZE";
- const char* stackVal = m_Makefile->GetDefinition(stackVar.c_str());
- if(stackVal)
- {
- fout << "\t\t\t\tStackReserveSize=\"" << stackVal << "\"";
- }
- fout << "/>\n";
- break;
+ fout << "\t\t\t\tAdditionalLibraryDirectories=\"";
+ this->OutputLibraryDirectories(fout, configName, libName, target);
+ fout << "\"\n";
+ fout << "\t\t\t\tProgramDatabaseFile=\"" << m_LibraryOutputPath
+ << "$(OutDir)\\" << libName << ".pdb\"\n";
+ if(strcmp(configName, "Debug") == 0
+ || strcmp(configName, "RelWithDebInfo") == 0)
+ {
+ fout << "\t\t\t\tGenerateDebugInformation=\"TRUE\"\n";
+ }
+ if ( target.GetPropertyAsBool("WIN32_EXECUTABLE") )
+ {
+ fout << "\t\t\t\tSubSystem=\"2\"\n";
+ }
+ else
+ {
+ fout << "\t\t\t\tSubSystem=\"1\"\n";
+ }
+ const char* linkLanguage = target.GetLinkerLanguage(this->GetGlobalGenerator());
+ if(!linkLanguage)
+ {
+ cmSystemTools::Error("CMake can not determine linker language for target:",
+ target.GetName());
+ return;
}
+ std::string stackVar = "CMAKE_";
+ stackVar += linkLanguage;
+ stackVar += "_STACK_SIZE";
+ const char* stackVal = m_Makefile->GetDefinition(stackVar.c_str());
+ if(stackVal)
+ {
+ fout << "\t\t\t\tStackReserveSize=\"" << stackVal << "\"";
+ }
+ fout << "/>\n";
+ break;
+ }
case cmTarget::UTILITY:
break;
}
}
void cmLocalVisualStudio7Generator::OutputModuleDefinitionFile(std::ostream& fout,
- cmTarget &target)
+ cmTarget &target)
{
std::vector<cmSourceFile*> const& classes = target.GetSourceFiles();
for(std::vector<cmSourceFile*>::const_iterator i = classes.begin();
@@ -812,9 +818,9 @@ void cmLocalVisualStudio7Generator::OutputModuleDefinitionFile(std::ostream& fou
}
void cmLocalVisualStudio7Generator::OutputLibraryDirectories(std::ostream& fout,
- const char*,
- const char*,
- cmTarget &tgt)
+ const char*,
+ const char*,
+ cmTarget &tgt)
{
bool hasone = false;
if(m_LibraryOutputPath.size())
@@ -865,9 +871,9 @@ void cmLocalVisualStudio7Generator::OutputLibraryDirectories(std::ostream& fout,
}
void cmLocalVisualStudio7Generator::OutputLibraries(std::ostream& fout,
- const char* configName,
- const char* libName,
- cmTarget &target)
+ const char* configName,
+ const char* libName,
+ cmTarget &target)
{
const cmTarget::LinkLibraries& libs = target.GetLinkLibraries();
cmTarget::LinkLibraries::const_iterator j;
@@ -988,135 +994,149 @@ void cmLocalVisualStudio7Generator::WriteVCProjFile(std::ostream& fout,
// Loop through every source group.
- for(std::vector<cmSourceGroup>::const_iterator sg = sourceGroups.begin();
- sg != sourceGroups.end(); ++sg)
+ for(unsigned int i = 0; i < sourceGroups.size(); ++i)
{
- const std::vector<const cmSourceFile *> &sourceFiles =
- sg->GetSourceFiles();
- // If the group is empty, don't write it at all.
- if(sourceFiles.empty())
- {
- continue;
- }
+ cmSourceGroup sg = sourceGroups[i];
+ this->WriteGroup(&sg, target, fout, libName, configs);
+ }
+
+ //}
- // If the group has a name, write the header.
- std::string name = sg->GetName();
- if(name != "")
+ fout << "\t</Files>\n";
+
+ // Write the VCProj file's footer.
+ this->WriteVCProjFooter(fout);
+}
+
+void cmLocalVisualStudio7Generator::WriteGroup(const cmSourceGroup *sg, cmTarget target, std::ostream &fout, const char *libName, std::vector<std::string> *configs)
+{
+ const std::vector<const cmSourceFile *> &sourceFiles =
+ sg->GetSourceFiles();
+ // If the group is empty, don't write it at all.
+ if(sourceFiles.empty())
+ {
+ return;
+ }
+
+ // If the group has a name, write the header.
+ std::string name = sg->GetName();
+ if(name != "")
+ {
+ this->WriteVCProjBeginGroup(fout, name.c_str(), "");
+ }
+
+ // Loop through each source in the source group.
+ for(std::vector<const cmSourceFile *>::const_iterator sf =
+ sourceFiles.begin(); sf != sourceFiles.end(); ++sf)
+ {
+ std::string source = (*sf)->GetFullPath();
+ const cmCustomCommand *command = (*sf)->GetCustomCommand();
+ std::string compileFlags;
+ std::string additionalDeps;
+
+ // Check for extra compiler flags.
+ const char* cflags = (*sf)->GetProperty("COMPILE_FLAGS");
+ if(cflags)
{
- this->WriteVCProjBeginGroup(fout, name.c_str(), "");
+ compileFlags = cflags;
}
-
- // Loop through each source in the source group.
- for(std::vector<const cmSourceFile *>::const_iterator sf =
- sourceFiles.begin(); sf != sourceFiles.end(); ++sf)
+ const char* lang =
+ m_GlobalGenerator->GetLanguageFromExtension((*sf)->GetSourceExtension().c_str());
+ if(lang && strcmp(lang, "CXX") == 0)
{
- std::string source = (*sf)->GetFullPath();
- const cmCustomCommand *command = (*sf)->GetCustomCommand();
- std::string compileFlags;
- std::string additionalDeps;
-
- // Check for extra compiler flags.
- const char* cflags = (*sf)->GetProperty("COMPILE_FLAGS");
- if(cflags)
+ // force a C++ file type
+ compileFlags += " /TP ";
+ }
+ // Check for extra object-file dependencies.
+ const char* deps = (*sf)->GetProperty("OBJECT_DEPENDS");
+ if(deps)
+ {
+ std::vector<std::string> depends;
+ cmSystemTools::ExpandListArgument(deps, depends);
+ if(!depends.empty())
{
- compileFlags = cflags;
+ std::vector<std::string>::iterator i = depends.begin();
+ additionalDeps = this->ConvertToXMLOutputPath(i->c_str());
+ for(++i;i != depends.end(); ++i)
+ {
+ additionalDeps += ";";
+ additionalDeps += this->ConvertToXMLOutputPath(i->c_str());
+ }
}
- const char* lang =
- m_GlobalGenerator->GetLanguageFromExtension((*sf)->GetSourceExtension().c_str());
- if(lang && strcmp(lang, "CXX") == 0)
+ }
+ if (source != libName || target.GetType() == cmTarget::UTILITY)
+ {
+ fout << "\t\t\t<File\n";
+ std::string d = this->ConvertToXMLOutputPathSingle(source.c_str());
+ // Tell MS-Dev what the source is. If the compiler knows how to
+ // build it, then it will.
+ fout << "\t\t\t\tRelativePath=\"" << d << "\">\n";
+ if (command)
{
- // force a C++ file type
- compileFlags += " /TP ";
+ // Construct the entire set of commands in one string.
+ std::string script = this->ConstructScript(command->GetCommandLines());
+ const char* comment = command->GetComment();
+ const char* flags = compileFlags.size() ? compileFlags.c_str(): 0;
+ this->WriteCustomRule(fout, source.c_str(), script.c_str(),
+ (*comment?comment:"Custom Rule"),
+ command->GetDepends(),
+ command->GetOutput(), flags);
}
- // Check for extra object-file dependencies.
- const char* deps = (*sf)->GetProperty("OBJECT_DEPENDS");
- if(deps)
+ else if(compileFlags.size() || additionalDeps.length())
{
- std::vector<std::string> depends;
- cmSystemTools::ExpandListArgument(deps, depends);
- if(!depends.empty())
+ const char* aCompilerTool = "VCCLCompilerTool";
+ std::string ext = (*sf)->GetSourceExtension();
+ ext = cmSystemTools::LowerCase(ext);
+ if(ext == "idl")
{
- std::vector<std::string>::iterator i = depends.begin();
- additionalDeps = this->ConvertToXMLOutputPath(i->c_str());
- for(++i;i != depends.end(); ++i)
- {
- additionalDeps += ";";
- additionalDeps += this->ConvertToXMLOutputPath(i->c_str());
- }
+ aCompilerTool = "VCMIDLTool";
}
- }
- if (source != libName || target.GetType() == cmTarget::UTILITY)
- {
- fout << "\t\t\t<File\n";
- std::string d = this->ConvertToXMLOutputPathSingle(source.c_str());
- // Tell MS-Dev what the source is. If the compiler knows how to
- // build it, then it will.
- fout << "\t\t\t\tRelativePath=\"" << d << "\">\n";
- if (command)
+ if(ext == "rc")
{
- // Construct the entire set of commands in one string.
- std::string script = this->ConstructScript(command->GetCommandLines());
- const char* comment = command->GetComment();
- const char* flags = compileFlags.size() ? compileFlags.c_str(): 0;
- this->WriteCustomRule(fout, source.c_str(), script.c_str(),
- (*comment?comment:"Custom Rule"),
- command->GetDepends(),
- command->GetOutput(), flags);
+ aCompilerTool = "VCResourceCompilerTool";
}
- else if(compileFlags.size() || additionalDeps.length())
+ if(ext == "def")
{
- const char* aCompilerTool = "VCCLCompilerTool";
- std::string ext = (*sf)->GetSourceExtension();
- ext = cmSystemTools::LowerCase(ext);
- if(ext == "idl")
- {
- aCompilerTool = "VCMIDLTool";
- }
- if(ext == "rc")
- {
- aCompilerTool = "VCResourceCompilerTool";
- }
- if(ext == "def")
+ aCompilerTool = "VCCustomBuildTool";
+ }
+ for(std::vector<std::string>::iterator i = configs->begin();
+ i != configs->end(); ++i)
+ {
+ fout << "\t\t\t\t<FileConfiguration\n"
+ << "\t\t\t\t\tName=\"" << *i << "|Win32\">\n"
+ << "\t\t\t\t\t<Tool\n"
+ << "\t\t\t\t\tName=\"" << aCompilerTool << "\"\n";
+ if(compileFlags.size())
{
- aCompilerTool = "VCCustomBuildTool";
+ fout << "\t\t\t\t\tAdditionalOptions=\""
+ << this->EscapeForXML(compileFlags.c_str()) << "\"\n";
}
- for(std::vector<std::string>::iterator i = configs->begin();
- i != configs->end(); ++i)
+ if(additionalDeps.length())
{
- fout << "\t\t\t\t<FileConfiguration\n"
- << "\t\t\t\t\tName=\"" << *i << "|Win32\">\n"
- << "\t\t\t\t\t<Tool\n"
- << "\t\t\t\t\tName=\"" << aCompilerTool << "\"\n";
- if(compileFlags.size())
- {
- fout << "\t\t\t\t\tAdditionalOptions=\""
- << this->EscapeForXML(compileFlags.c_str()) << "\"\n";
- }
- if(additionalDeps.length())
- {
- fout << "\t\t\t\t\tAdditionalDependencies=\""
- << additionalDeps.c_str() << "\"\n";
- }
- fout << "\t\t\t\t\t/>\n"
- << "\t\t\t\t</FileConfiguration>\n";
+ fout << "\t\t\t\t\tAdditionalDependencies=\""
+ << additionalDeps.c_str() << "\"\n";
}
+ fout << "\t\t\t\t\t/>\n"
+ << "\t\t\t\t</FileConfiguration>\n";
}
- fout << "\t\t\t</File>\n";
}
+ fout << "\t\t\t</File>\n";
}
-
- // If the group has a name, write the footer.
- if(name != "")
- {
- this->WriteVCProjEndGroup(fout);
- }
- }
- fout << "\t</Files>\n";
-
- // Write the VCProj file's footer.
- this->WriteVCProjFooter(fout);
-}
+ }
+
+ std::vector<cmSourceGroup> children = sg->GetGroupChildren();
+ for(unsigned int i=0;i<children.size();++i)
+ {
+ this->WriteGroup(&children[i], target, fout, libName, configs);
+ }
+
+ // If the group has a name, write the footer.
+ if(name != "")
+ {
+ this->WriteVCProjEndGroup(fout);
+ }
+}
void cmLocalVisualStudio7Generator::
WriteCustomRule(std::ostream& fout,
@@ -1172,7 +1192,7 @@ WriteCustomRule(std::ostream& fout,
}
if(exePath.size())
{
- libPath = exePath;
+ libPath = exePath;
}
else
{
@@ -1207,8 +1227,8 @@ WriteCustomRule(std::ostream& fout,
void cmLocalVisualStudio7Generator::WriteVCProjBeginGroup(std::ostream& fout,
- const char* group,
- const char* )
+ const char* group,
+ const char* )
{
fout << "\t\t<Filter\n"
<< "\t\t\tName=\"" << group << "\"\n"
diff --git a/Source/cmLocalVisualStudio7Generator.h b/Source/cmLocalVisualStudio7Generator.h
index c8cd0ed..af35274 100644
--- a/Source/cmLocalVisualStudio7Generator.h
+++ b/Source/cmLocalVisualStudio7Generator.h
@@ -111,6 +111,8 @@ private:
const char* output,
const char* extraFlags);
+ void WriteGroup(const cmSourceGroup *sg, cmTarget target, std::ostream &fout, const char *libName, std::vector<std::string> *configs);
+
std::vector<std::string> m_CreatedProjectNames;
std::string m_LibraryOutputPath;
std::string m_ExecutableOutputPath;
diff --git a/Source/cmMakefile.cxx b/Source/cmMakefile.cxx
index f4cd035..1bd2115 100644
--- a/Source/cmMakefile.cxx
+++ b/Source/cmMakefile.cxx
@@ -241,8 +241,8 @@ bool cmMakefile::ExecuteCommand(const cmListFileFunction& lff)
{
cmOStringStream error;
error << "Error in cmake code at\n"
- << lff.m_FilePath << ":" << lff.m_Line << ":\n"
- << rm->GetError();
+ << lff.m_FilePath << ":" << lff.m_Line << ":\n"
+ << rm->GetError();
cmSystemTools::Error(error.str().c_str());
return false;
}
@@ -250,8 +250,8 @@ bool cmMakefile::ExecuteCommand(const cmListFileFunction& lff)
usedCommand->SetMakefile(this);
bool keepCommand = false;
if(usedCommand->GetEnabled() && !cmSystemTools::GetFatalErrorOccured() &&
- (!this->GetCMakeInstance()->GetScriptMode() ||
- usedCommand->IsScriptable()))
+ (!this->GetCMakeInstance()->GetScriptMode() ||
+ usedCommand->IsScriptable()))
{
if(!usedCommand->InvokeInitialPass(lff.m_Arguments))
{
@@ -277,8 +277,8 @@ bool cmMakefile::ExecuteCommand(const cmListFileFunction& lff)
{
cmOStringStream error;
error << "Error in cmake code at\n"
- << lff.m_FilePath << ":" << lff.m_Line << ":\n"
- << "Command " << usedCommand->GetName() << " not scriptable" << std::endl;
+ << lff.m_FilePath << ":" << lff.m_Line << ":\n"
+ << "Command " << usedCommand->GetName() << " not scriptable" << std::endl;
cmSystemTools::Error(error.str().c_str());
result = false;
cmSystemTools::SetFatalErrorOccured();
@@ -367,7 +367,7 @@ bool cmMakefile::ReadListFile(const char* filename_in, const char *external_in)
if(!external && m_cmStartDirectory == m_cmHomeDirectory)
{
if(cmSystemTools::LowerCase(
- cmSystemTools::GetFilenameName(filename)) == "cmakelists.txt")
+ cmSystemTools::GetFilenameName(filename)) == "cmakelists.txt")
{
requireProjectCommand = true;
}
@@ -418,7 +418,7 @@ void cmMakefile::AddCommand(cmCommand* wg)
this->GetCMakeInstance()->AddCommand(wg);
}
- // Set the make file
+// Set the make file
void cmMakefile::SetLocalGenerator(cmLocalGenerator* lg)
{
m_LocalGenerator = lg;
@@ -439,7 +439,7 @@ void cmMakefile::FinalPass()
}
- // Generate the output file
+// Generate the output file
void cmMakefile::ConfigureFinalPass()
{
this->FinalPass();
@@ -725,7 +725,7 @@ void cmMakefile::RemoveDefineFlag(const char* flag)
void cmMakefile::AddLinkLibrary(const char* lib, cmTarget::LinkLibraryType llt)
{
m_LinkLibraries.push_back(
- std::pair<std::string, cmTarget::LinkLibraryType>(lib,llt));
+ std::pair<std::string, cmTarget::LinkLibraryType>(lib,llt));
}
void cmMakefile::AddLinkLibraryForTarget(const char *target,
@@ -892,7 +892,7 @@ void cmMakefile::AddIncludeDirectory(const char* inc, bool before)
// order dependency of the include path.
std::vector<std::string>::iterator i =
std::find(m_IncludeDirectories.begin(),
- m_IncludeDirectories.end(), inc);
+ m_IncludeDirectories.end(), inc);
if(i == m_IncludeDirectories.end())
{
if (before)
@@ -945,7 +945,7 @@ void cmMakefile::AddCacheDefinition(const char* name, const char* value,
cmCacheManager::CacheIterator it =
this->GetCacheManager()->GetCacheIterator(name);
if(!it.IsAtEnd() && (it.GetType() == cmCacheManager::UNINITIALIZED) &&
- it.Initialized())
+ it.Initialized())
{
val = it.GetValue();
if ( type == cmCacheManager::PATH || type == cmCacheManager::FILEPATH )
@@ -1002,7 +1002,7 @@ void cmMakefile::AddCacheDefinition(const char* name, bool value, const char* do
cmCacheManager::CacheIterator it =
this->GetCacheManager()->GetCacheIterator(name);
if(!it.IsAtEnd() && (it.GetType() == cmCacheManager::UNINITIALIZED) &&
- it.Initialized())
+ it.Initialized())
{
val = it.GetValueAsBool();
}
@@ -1093,8 +1093,8 @@ void cmMakefile::AddLibrary(const char* lname, int shared,
{
case 0:
this->GetCacheManager()->AddCacheEntry(ltname.c_str(),"STATIC",
- "Whether a library is static, shared or module.",
- cmCacheManager::INTERNAL);
+ "Whether a library is static, shared or module.",
+ cmCacheManager::INTERNAL);
break;
case 1:
this->GetCacheManager()->
@@ -1120,7 +1120,7 @@ void cmMakefile::AddLibrary(const char* lname, int shared,
}
cmTarget* cmMakefile::AddExecutable(const char *exeName,
- const std::vector<std::string> &srcs)
+ const std::vector<std::string> &srcs)
{
cmTarget target;
target.SetType(cmTarget::EXECUTABLE, exeName);
@@ -1180,30 +1180,88 @@ cmSourceGroup* cmMakefile::GetSourceGroup(const char* name)
if(sgName == name)
{
return &(*sg);
+ }
+ else
+ {
+ cmSourceGroup *target = sg->lookupChild(name);
+
+ if(target)
+ {
+ return target;
+ }
}
}
return 0;
}
-void cmMakefile::AddSourceGroup(const char* name, const char* regex)
+void cmMakefile::AddSourceGroup(const char* name, const char* regex, const char *parent)
{
// First see if the group exists. If so, replace its regular expression.
- for(std::vector<cmSourceGroup>::iterator sg = m_SourceGroups.begin();
- sg != m_SourceGroups.end(); ++sg)
+ for(unsigned int i=0;i<m_SourceGroups.size();++i)
{
+ cmSourceGroup *sg = &m_SourceGroups[i];
+
std::string sgName = sg->GetName();
- if(sgName == name)
+ if(!parent)
{
- if ( regex )
+ if(sgName == name)
{
- // We only want to set the regular expression. If there are already
- // source files in the group, we don't want to remove them.
- sg->SetGroupRegex(regex);
+ if ( regex )
+ {
+ // We only want to set the regular expression. If there are already
+ // source files in the group, we don't want to remove them.
+ sg->SetGroupRegex(regex);
+ }
+ return;
+ }
+ }
+ else
+ {
+ if(sgName == parent)
+ {
+ cmSourceGroup *localtarget = sg->lookupChild(name);
+ if(localtarget)
+ {
+ if ( regex )
+ {
+ // We only want to set the regular expression. If there are already
+ // source files in the group, we don't want to remove them.
+ localtarget->SetGroupRegex(regex);
+ }
+ }
+ else
+ {
+ sg->AddChild(cmSourceGroup(name, regex));
+ }
+ return;
+ }
+ else
+ {
+ cmSourceGroup *localtarget = sg->lookupChild(parent);
+
+ if(localtarget)
+ {
+ cmSourceGroup *addtarget = localtarget->lookupChild(name);
+
+ if(addtarget)
+ {
+ if ( regex )
+ {
+ // We only want to set the regular expression. If there are already
+ // source files in the group, we don't want to remove them.
+ addtarget->SetGroupRegex(regex);
+ }
+ }
+ else
+ {
+ localtarget->AddChild(cmSourceGroup(name, regex));
+ }
+ return;
+ }
}
- return;
}
}
-
+
// The group doesn't exist. Add it.
m_SourceGroups.push_back(cmSourceGroup(name, regex));
}
@@ -1221,12 +1279,12 @@ void cmMakefile::ExpandVariables()
{
// Now expand variables in the include and link strings
for(std::vector<std::string>::iterator d = m_IncludeDirectories.begin();
- d != m_IncludeDirectories.end(); ++d)
+ d != m_IncludeDirectories.end(); ++d)
{
this->ExpandVariablesInString(*d);
}
for(std::vector<std::string>::iterator d = m_LinkDirectories.begin();
- d != m_LinkDirectories.end(); ++d)
+ d != m_LinkDirectories.end(); ++d)
{
this->ExpandVariablesInString(*d);
}
@@ -1426,158 +1484,158 @@ const char *cmMakefile::ExpandVariablesInString(std::string& source,
if ( notParsed )
{
- // start by look for $ or @ in the string
- std::string::size_type markerPos;
- if(atOnly)
- {
- markerPos = source.find_first_of("@");
- }
- else
- {
- markerPos = source.find_first_of("$@");
- }
- // if not found, or found as the last character, then leave quickly as
- // nothing needs to be expanded
- if((markerPos == std::string::npos) || (markerPos >= source.size()-1))
- {
- return source.c_str();
- }
- // current position
- std::string::size_type currentPos =0; // start at 0
- std::string result; // string with replacements
- // go until the the end of the string
- while((markerPos != std::string::npos) && (markerPos < source.size()-1))
- {
- // grab string from currentPos to the start of the variable
- // and add it to the result
- result += source.substr(currentPos, markerPos - currentPos);
- char endVariableMarker; // what is the end of the variable @ or }
- int markerStartSize = 1; // size of the start marker 1 or 2 or 5
- if(!atOnly && source[markerPos] == '$')
- {
- // ${var} case
- if(source[markerPos+1] == '{')
- {
- endVariableMarker = '}';
- markerStartSize = 2;
- }
- // $ENV{var} case
- else if(markerPos+4 < source.size() &&
- source[markerPos+4] == '{' &&
- !source.substr(markerPos+1, 3).compare("ENV"))
- {
- endVariableMarker = '}';
- markerStartSize = 5;
- }
- else
- {
- // bogus $ with no { so add $ to result and move on
- result += '$'; // add bogus $ back into string
- currentPos = markerPos+1; // move on
- endVariableMarker = ' '; // set end var to space so we can tell bogus
- }
+ // start by look for $ or @ in the string
+ std::string::size_type markerPos;
+ if(atOnly)
+ {
+ markerPos = source.find_first_of("@");
}
else
{
- // @VAR case
- endVariableMarker = '@';
+ markerPos = source.find_first_of("$@");
+ }
+ // if not found, or found as the last character, then leave quickly as
+ // nothing needs to be expanded
+ if((markerPos == std::string::npos) || (markerPos >= source.size()-1))
+ {
+ return source.c_str();
}
- // if it was a valid variable (started with @ or ${ or $ENV{ )
- if(endVariableMarker != ' ')
+ // current position
+ std::string::size_type currentPos =0; // start at 0
+ std::string result; // string with replacements
+ // go until the the end of the string
+ while((markerPos != std::string::npos) && (markerPos < source.size()-1))
{
- markerPos += markerStartSize; // move past marker
- // find the end variable marker starting at the markerPos
- std::string::size_type endVariablePos =
- source.find(endVariableMarker, markerPos);
- if(endVariablePos == std::string::npos)
+ // grab string from currentPos to the start of the variable
+ // and add it to the result
+ result += source.substr(currentPos, markerPos - currentPos);
+ char endVariableMarker; // what is the end of the variable @ or }
+ int markerStartSize = 1; // size of the start marker 1 or 2 or 5
+ if(!atOnly && source[markerPos] == '$')
{
- // no end marker found so add the bogus start
- if(endVariableMarker == '@')
+ // ${var} case
+ if(source[markerPos+1] == '{')
{
- result += '@';
+ endVariableMarker = '}';
+ markerStartSize = 2;
+ }
+ // $ENV{var} case
+ else if(markerPos+4 < source.size() &&
+ source[markerPos+4] == '{' &&
+ !source.substr(markerPos+1, 3).compare("ENV"))
+ {
+ endVariableMarker = '}';
+ markerStartSize = 5;
}
else
{
- result += (markerStartSize == 5 ? "$ENV{" : "${");
+ // bogus $ with no { so add $ to result and move on
+ result += '$'; // add bogus $ back into string
+ currentPos = markerPos+1; // move on
+ endVariableMarker = ' '; // set end var to space so we can tell bogus
}
- currentPos = markerPos;
}
else
{
- // good variable remove it
- std::string var = source.substr(markerPos, endVariablePos - markerPos);
- bool found = false;
- if (markerStartSize == 5) // $ENV{
+ // @VAR case
+ endVariableMarker = '@';
+ }
+ // if it was a valid variable (started with @ or ${ or $ENV{ )
+ if(endVariableMarker != ' ')
+ {
+ markerPos += markerStartSize; // move past marker
+ // find the end variable marker starting at the markerPos
+ std::string::size_type endVariablePos =
+ source.find(endVariableMarker, markerPos);
+ if(endVariablePos == std::string::npos)
{
- char *ptr = getenv(var.c_str());
- if (ptr)
+ // no end marker found so add the bogus start
+ if(endVariableMarker == '@')
{
- if (escapeQuotes)
- {
- result += cmSystemTools::EscapeQuotes(ptr);
- }
- else
- {
- result += ptr;
- }
- found = true;
+ result += '@';
+ }
+ else
+ {
+ result += (markerStartSize == 5 ? "$ENV{" : "${");
}
+ currentPos = markerPos;
}
else
{
- const char* lookup = this->GetDefinition(var.c_str());
- if(lookup)
+ // good variable remove it
+ std::string var = source.substr(markerPos, endVariablePos - markerPos);
+ bool found = false;
+ if (markerStartSize == 5) // $ENV{
{
- if (escapeQuotes)
- {
- result += cmSystemTools::EscapeQuotes(lookup);
- }
- else
+ char *ptr = getenv(var.c_str());
+ if (ptr)
{
- result += lookup;
+ if (escapeQuotes)
+ {
+ result += cmSystemTools::EscapeQuotes(ptr);
+ }
+ else
+ {
+ result += ptr;
+ }
+ found = true;
}
- found = true;
}
- else if(filename && (var == "CMAKE_CURRENT_LIST_FILE"))
+ else
{
- result += filename;
- found = true;
- }
- else if(line >= 0 && (var == "CMAKE_CURRENT_LIST_LINE"))
- {
- cmOStringStream ostr;
- ostr << line;
- result += ostr.str();
- found = true;
+ const char* lookup = this->GetDefinition(var.c_str());
+ if(lookup)
+ {
+ if (escapeQuotes)
+ {
+ result += cmSystemTools::EscapeQuotes(lookup);
+ }
+ else
+ {
+ result += lookup;
+ }
+ found = true;
+ }
+ else if(filename && (var == "CMAKE_CURRENT_LIST_FILE"))
+ {
+ result += filename;
+ found = true;
+ }
+ else if(line >= 0 && (var == "CMAKE_CURRENT_LIST_LINE"))
+ {
+ cmOStringStream ostr;
+ ostr << line;
+ result += ostr.str();
+ found = true;
+ }
}
- }
- // if found add to result, if not, then it gets blanked
- if (!found)
- {
- // if no definition is found then add the var back
- if(!removeEmpty && endVariableMarker == '@')
+ // if found add to result, if not, then it gets blanked
+ if (!found)
{
- result += "@";
- result += var;
- result += "@";
+ // if no definition is found then add the var back
+ if(!removeEmpty && endVariableMarker == '@')
+ {
+ result += "@";
+ result += var;
+ result += "@";
+ }
}
+ // lookup var, and replace it
+ currentPos = endVariablePos+1;
}
- // lookup var, and replace it
- currentPos = endVariablePos+1;
+ }
+ if(atOnly)
+ {
+ markerPos = source.find_first_of("@", currentPos);
+ }
+ else
+ {
+ markerPos = source.find_first_of("$@", currentPos);
}
}
- if(atOnly)
- {
- markerPos = source.find_first_of("@", currentPos);
- }
- else
- {
- markerPos = source.find_first_of("$@", currentPos);
- }
+ result += source.substr(currentPos); // pick up the rest of the string
+ source = result;
}
- result += source.substr(currentPos); // pick up the rest of the string
- source = result;
- }
return source.c_str();
}
@@ -1651,9 +1709,10 @@ cmMakefile::FindSourceGroup(const char* source,
for(std::vector<cmSourceGroup>::reverse_iterator sg = groups.rbegin();
sg != groups.rend(); ++sg)
{
- if(sg->MatchesFiles(source))
+ cmSourceGroup *result = sg->MatchChildrenFiles(source);
+ if(result)
{
- return *sg;
+ return *result;
}
}
@@ -1661,12 +1720,14 @@ cmMakefile::FindSourceGroup(const char* source,
for(std::vector<cmSourceGroup>::reverse_iterator sg = groups.rbegin();
sg != groups.rend(); ++sg)
{
- if(sg->MatchesRegex(source))
+ cmSourceGroup *result = sg->MatchChildrenRegex(source);
+ if(result)
{
- return *sg;
+ return *result;
}
}
-
+
+
// Shouldn't get here, but just in case, return the default group.
return groups.front();
}
@@ -1914,12 +1975,12 @@ cmSourceFile* cmMakefile::GetOrCreateSource(const char* sourceName,
if ( cmSystemTools::GetFilenameLastExtension(srcTreeFile.c_str()).size() == 0)
{
if (cmSystemTools::DoesFileExistWithExtensions(
- srcTreeFile.c_str(), this->GetSourceExtensions()))
+ srcTreeFile.c_str(), this->GetSourceExtensions()))
{
src = srcTreeFile;
}
else if (cmSystemTools::DoesFileExistWithExtensions(
- srcTreeFile.c_str(), this->GetHeaderExtensions()))
+ srcTreeFile.c_str(), this->GetHeaderExtensions()))
{
src = srcTreeFile;
}
@@ -2288,7 +2349,7 @@ std::string cmMakefile::GetModulesFile(const char* filename)
}
//std::string Look through the possible module directories.
for(std::vector<std::string>::iterator i = modulePath.begin();
- i != modulePath.end(); ++i)
+ i != modulePath.end(); ++i)
{
std::string itempl = *i;
cmSystemTools::ConvertToUnixSlashes(itempl);
@@ -2364,7 +2425,7 @@ void cmMakefile::ConfigureString(const std::string& input,
}
int cmMakefile::ConfigureFile(const char* infile, const char* outfile,
- bool copyonly, bool atOnly, bool escapeQuotes)
+ bool copyonly, bool atOnly, bool escapeQuotes)
{
int res = 1;
if ( !cmSystemTools::FileExists(infile) )
@@ -2388,7 +2449,7 @@ int cmMakefile::ConfigureFile(const char* infile, const char* outfile,
if(copyonly)
{
if ( !cmSystemTools::CopyFileIfDifferent(sinfile.c_str(),
- soutfile.c_str()))
+ soutfile.c_str()))
{
return 0;
}
@@ -2402,7 +2463,7 @@ int cmMakefile::ConfigureFile(const char* infile, const char* outfile,
{
cmSystemTools::Error(
"Could not open file for write in copy operation ",
- tempOutputFile.c_str());
+ tempOutputFile.c_str());
cmSystemTools::ReportLastSystemError("");
return 0;
}
@@ -2428,7 +2489,7 @@ int cmMakefile::ConfigureFile(const char* infile, const char* outfile,
fin.close();
fout.close();
if ( !cmSystemTools::CopyFileIfDifferent(tempOutputFile.c_str(),
- soutfile.c_str()) )
+ soutfile.c_str()) )
{
res = 0;
}
@@ -2451,8 +2512,8 @@ bool cmMakefile::CheckInfiniteLoops()
{
std::vector<std::string>::iterator it;
for ( it = m_ListFiles.begin();
- it != m_ListFiles.end();
- ++ it )
+ it != m_ListFiles.end();
+ ++ it )
{
if ( this->HasWrittenFile(it->c_str()) )
{
diff --git a/Source/cmMakefile.h b/Source/cmMakefile.h
index a6d42f4..000133e 100644
--- a/Source/cmMakefile.h
+++ b/Source/cmMakefile.h
@@ -162,7 +162,7 @@ public:
* Add an executable to the build.
*/
cmTarget* AddExecutable(const char *exename,
- const std::vector<std::string> &srcs);
+ const std::vector<std::string> &srcs);
/**
* Add a utility to the build. A utiltity target is a command that
@@ -200,15 +200,15 @@ public:
*/
std::vector<std::string>& GetLinkDirectories()
{
- return m_LinkDirectories;
+ return m_LinkDirectories;
}
const std::vector<std::string>& GetLinkDirectories() const
{
- return m_LinkDirectories;
+ return m_LinkDirectories;
}
void SetLinkDirectories(const std::vector<std::string>& vec)
{
- m_LinkDirectories = vec;
+ m_LinkDirectories = vec;
}
/**
@@ -268,7 +268,7 @@ public:
*/
const char* GetProjectName()
{
- return m_ProjectName.c_str();
+ return m_ProjectName.c_str();
}
/**
@@ -280,7 +280,7 @@ public:
/**
* Add a source group for consideration when adding a new source.
*/
- void AddSourceGroup(const char* name, const char* regex=0);
+ void AddSourceGroup(const char* name, const char* regex=0, const char* parent=0);
/**
* Add an auxiliary directory to the build.
@@ -310,12 +310,12 @@ public:
void SetHomeDirectory(const char* dir);
const char* GetHomeDirectory() const
{
- return m_cmHomeDirectory.c_str();
+ return m_cmHomeDirectory.c_str();
}
void SetHomeOutputDirectory(const char* lib);
const char* GetHomeOutputDirectory() const
{
- return m_HomeOutputDirectory.c_str();
+ return m_HomeOutputDirectory.c_str();
}
//@}
@@ -394,7 +394,7 @@ public:
}
const char* GetComplainRegularExpression()
{
- return m_ComplainFileRegularExpression.c_str();
+ return m_ComplainFileRegularExpression.c_str();
}
/**
@@ -409,15 +409,15 @@ public:
*/
std::vector<std::string>& GetIncludeDirectories()
{
- return m_IncludeDirectories;
+ return m_IncludeDirectories;
}
const std::vector<std::string>& GetIncludeDirectories() const
{
- return m_IncludeDirectories;
+ return m_IncludeDirectories;
}
void SetIncludeDirectories(const std::vector<std::string>& vec)
{
- m_IncludeDirectories = vec;
+ m_IncludeDirectories = vec;
}
/** Expand out any arguements in the vector that have ; separated
@@ -554,7 +554,7 @@ public:
* Copy file but change lines acording to ConfigureString
*/
int ConfigureFile(const char* infile, const char* outfile,
- bool copyonly, bool atOnly, bool escapeQuotes);
+ bool copyonly, bool atOnly, bool escapeQuotes);
/**
* find what source group this source is in
diff --git a/Source/cmSourceGroup.cxx b/Source/cmSourceGroup.cxx
index 035b643..58069d1 100644
--- a/Source/cmSourceGroup.cxx
+++ b/Source/cmSourceGroup.cxx
@@ -81,3 +81,88 @@ std::vector<const cmSourceFile*>& cmSourceGroup::GetSourceFiles()
{
return m_SourceFiles;
}
+
+//----------------------------------------------------------------------------
+void cmSourceGroup::AddChild(cmSourceGroup child)
+{
+ m_GroupChildren.push_back(child);
+}
+
+//----------------------------------------------------------------------------
+cmSourceGroup *cmSourceGroup::lookupChild(const char* name)
+{
+ // initializing iterators
+ std::vector<cmSourceGroup>::iterator iter = m_GroupChildren.begin();
+ std::vector<cmSourceGroup>::iterator end = m_GroupChildren.end();
+
+ // st
+ for(;iter!=end; ++iter)
+ {
+ std::string sgName = iter->GetName();
+
+ // look if descenened is the one were looking for
+ if(sgName == name)
+ {
+ return &(*iter); // if it so return it
+ }
+ // if the descendend isn't the one where looking for ask it's traverse
+ cmSourceGroup *result = iter->lookupChild(name);
+
+ // if one of it's descendeds is the one we're looking for return it
+ if(result)
+ {
+ return result;
+ }
+ }
+
+ // if no child with this name was found return NULL
+ return NULL;
+}
+
+cmSourceGroup *cmSourceGroup::MatchChildrenFiles(const char *name)
+{
+ // initializing iterators
+ std::vector<cmSourceGroup>::iterator iter = m_GroupChildren.begin();
+ std::vector<cmSourceGroup>::iterator end = m_GroupChildren.end();
+
+ if(this->MatchesFiles(name))
+ {
+ return this;
+ }
+ for(;iter!=end;++iter)
+ {
+ cmSourceGroup *result = iter->MatchChildrenFiles(name);
+ if(result)
+ {
+ return result;
+ }
+ }
+ return 0;
+}
+
+
+cmSourceGroup *cmSourceGroup::MatchChildrenRegex(const char *name)
+{
+ // initializing iterators
+ std::vector<cmSourceGroup>::iterator iter = m_GroupChildren.begin();
+ std::vector<cmSourceGroup>::iterator end = m_GroupChildren.end();
+
+ if(this->MatchesRegex(name))
+ {
+ return this;
+ }
+ for(;iter!=end; ++iter)
+ {
+ cmSourceGroup *result = iter->MatchChildrenRegex(name);
+ if(result)
+ {
+ return result;
+ }
+ }
+ return 0;
+}
+
+std::vector<cmSourceGroup> cmSourceGroup::GetGroupChildren() const
+{
+ return m_GroupChildren;
+}
diff --git a/Source/cmSourceGroup.h b/Source/cmSourceGroup.h
index 4308028..6b76bac 100644
--- a/Source/cmSourceGroup.h
+++ b/Source/cmSourceGroup.h
@@ -47,6 +47,16 @@ public:
* Add a file name to the explicit list of files for this group.
*/
void AddGroupFile(const char* name);
+
+ /**
+ * Add child to this sourcegroup
+ */
+ void AddChild(cmSourceGroup child);
+
+ /**
+ * Looks up child and returns it
+ */
+ cmSourceGroup *lookupChild(const char *name);
/**
* Get the name of this group.
@@ -62,6 +72,16 @@ public:
* Check if the given name matches this group's explicit file list.
*/
bool MatchesFiles(const char* name);
+
+ /**
+ * Check if the given name matches this group's explicit file list in children.
+ */
+ cmSourceGroup *MatchChildrenFiles(const char *name);
+
+ /**
+ * Check if the given name matches this group's regex in children.
+ */
+ cmSourceGroup *MatchChildrenRegex(const char *name);
/**
* Assign the given source file to this group. Used only by
@@ -76,6 +96,7 @@ public:
const std::vector<const cmSourceFile*>& GetSourceFiles() const;
std::vector<const cmSourceFile*>& GetSourceFiles();
+ std::vector<cmSourceGroup> GetGroupChildren() const;
private:
/**
* The name of the source group.
@@ -97,6 +118,8 @@ private:
* this group.
*/
std::vector<const cmSourceFile*> m_SourceFiles;
+
+ std::vector<cmSourceGroup> m_GroupChildren;
};
#endif
diff --git a/Source/cmSourceGroupCommand.cxx b/Source/cmSourceGroupCommand.cxx
index 131e86c..01b969f 100644
--- a/Source/cmSourceGroupCommand.cxx
+++ b/Source/cmSourceGroupCommand.cxx
@@ -16,6 +16,50 @@
=========================================================================*/
#include "cmSourceGroupCommand.h"
+inline std::vector<std::string> tokenize(const std::string& str,
+ const std::string& sep,
+ bool skipEmptyTokens)
+{
+ std::vector<std::string> tokens;
+ std::string::size_type tokstart,tokend;
+
+ if (skipEmptyTokens)
+ {
+ tokend=0;
+ }
+ else
+ {
+ tokend=std::string::npos;
+ }
+
+ do
+ {
+ if (skipEmptyTokens)
+ {
+ tokstart=str.find_first_not_of(sep,tokend);
+ }
+ else
+ {
+ tokstart=tokend+1;
+ }
+ if (tokstart==std::string::npos)
+ {
+ break; // no more tokens
+ }
+ tokend=str.find_first_of(sep,tokstart);
+ if (tokend==std::string::npos)
+ {
+ tokens.push_back(str.substr(tokstart));
+ }
+ else
+ {
+ tokens.push_back(str.substr(tokstart,tokend-tokstart));
+ }
+ } while (tokend!=std::string::npos);
+
+ return tokens;
+}
+
// cmSourceGroupCommand
bool cmSourceGroupCommand::InitialPass(std::vector<std::string> const& args)
{
@@ -24,18 +68,32 @@ bool cmSourceGroupCommand::InitialPass(std::vector<std::string> const& args)
this->SetError("called with incorrect number of arguments");
return false;
}
-
- // Get the source group with the given name.
- cmSourceGroup* sg = m_Makefile->GetSourceGroup(args[0].c_str());
- if(!sg)
+
+ std::string delimiter = "\\";
+
+ if(m_Makefile->GetDefinition("SOURCE_GROUP_DELIMITER"))
+ delimiter = m_Makefile->GetDefinition("SOURCE_GROUP_DELIMITER");
+
+ std::vector<std::string> folders = tokenize(args[0], delimiter, true);
+
+ const char *parent = NULL;
+
+ cmSourceGroup* sg = NULL;
+
+ for(unsigned int i=0;i<folders.size();++i)
{
- m_Makefile->AddSourceGroup(args[0].c_str(), 0);
- sg = m_Makefile->GetSourceGroup(args[0].c_str());
+ sg = m_Makefile->GetSourceGroup(folders[i].c_str());
+ if(!sg)
+ {
+ m_Makefile->AddSourceGroup(folders[i].c_str(), 0, parent);
+ }
+ sg = m_Makefile->GetSourceGroup(folders[i].c_str());
+ parent = folders[i].c_str();
}
-
+
// If only two arguments are given, the pre-1.8 version of the
// command is being invoked.
- if(args.size() == 2 && args[1] != "FILES")
+ if(args.size() == 2 && args[1] != "FILES")
{
sg->SetGroupRegex(args[1].c_str());
return true;