From 754485afff2b0dcf6ba16af8b65d08d1fcf75702 Mon Sep 17 00:00:00 2001 From: Daniel Pfeifer Date: Sun, 2 Aug 2015 22:06:38 +0200 Subject: cmExtraEclipseCDT4Generator: port to cmXMLWriter --- Source/cmExtraEclipseCDT4Generator.cxx | 687 +++++++++++++++------------------ Source/cmExtraEclipseCDT4Generator.h | 21 +- 2 files changed, 328 insertions(+), 380 deletions(-) diff --git a/Source/cmExtraEclipseCDT4Generator.cxx b/Source/cmExtraEclipseCDT4Generator.cxx index 133a85a..f0227b0 100644 --- a/Source/cmExtraEclipseCDT4Generator.cxx +++ b/Source/cmExtraEclipseCDT4Generator.cxx @@ -19,11 +19,29 @@ #include "cmState.h" #include "cmTarget.h" #include "cmSourceFile.h" +#include "cmXMLWriter.h" #include "cmSystemTools.h" #include #include +static void AppendAttribute(cmXMLWriter& xml, const char* keyval) +{ + xml.StartElement("attribute"); + xml.Attribute("key", keyval); + xml.Attribute("value", keyval); + xml.EndElement(); +} + +template +void AppendDictionary(cmXMLWriter& xml, const char* key, T const& value) +{ + xml.StartElement("dictionary"); + xml.Element("key", key); + xml.Element("value", value); + xml.EndElement(); +} + //---------------------------------------------------------------------------- cmExtraEclipseCDT4Generator ::cmExtraEclipseCDT4Generator() : cmExternalMakefileProjectGenerator() @@ -172,34 +190,30 @@ void cmExtraEclipseCDT4Generator::CreateSourceProjectFile() return; } - fout << - "\n" - "\n" - "\t" << this->EscapeForXML(name) << "\n" - "\t\n" - "\t\n" - "\t\n" - "\t\n" - "\t\n" - "\t\n" - "\t\n" - "\t\n"; + cmXMLWriter xml(fout); + xml.StartDocument("UTF-8"); + xml.StartElement("projectDescription"); + xml.Element("name", name); + xml.Element("comment", ""); + xml.Element("projects", ""); + xml.Element("buildSpec", ""); + xml.Element("natures", ""); + xml.StartElement("linkedResources"); if (this->SupportsVirtualFolders) { - this->CreateLinksToSubprojects(fout, this->HomeDirectory); + this->CreateLinksToSubprojects(xml, this->HomeDirectory); this->SrcLinkedResources.clear(); } - fout << - "\t\n" - "\n" - ; + xml.EndElement(); // linkedResources + xml.EndElement(); // projectDescription + xml.EndDocument(); } //---------------------------------------------------------------------------- -void cmExtraEclipseCDT4Generator::AddEnvVar(cmGeneratedFileStream& fout, +void cmExtraEclipseCDT4Generator::AddEnvVar(std::ostream& out, const char* envVar, cmLocalGenerator* lg) { @@ -257,7 +271,7 @@ void cmExtraEclipseCDT4Generator::AddEnvVar(cmGeneratedFileStream& fout, if (!valueToUse.empty()) { - fout << envVar << "=" << valueToUse << "|"; + out << envVar << "=" << valueToUse << "|"; } } @@ -282,197 +296,125 @@ void cmExtraEclipseCDT4Generator::CreateProjectFile() compilerId = mf->GetSafeDefinition("CMAKE_CXX_COMPILER_ID"); } - fout << - "\n" - "\n" - "\t" << - this->GenerateProjectName(lg->GetProjectName(), - mf->GetSafeDefinition("CMAKE_BUILD_TYPE"), - this->GetPathBasename(this->HomeOutputDirectory)) - << "\n" - "\t\n" - "\t\n" - "\t\n" - "\t\n" - "\t\t\n" - "\t\t\torg.eclipse.cdt.make.core.makeBuilder\n" - "\t\t\tclean,full,incremental,\n" - "\t\t\t\n" - ; + cmXMLWriter xml(fout); + + xml.StartDocument("UTF-8"); + xml.StartElement("projectDescription"); + + xml.Element("name", this->GenerateProjectName(lg->GetProjectName(), + mf->GetSafeDefinition("CMAKE_BUILD_TYPE"), + this->GetPathBasename(this->HomeOutputDirectory))); + + xml.Element("comment", ""); + xml.Element("projects", ""); + + xml.StartElement("buildSpec"); + xml.StartElement("buildCommand"); + xml.Element("name", "org.eclipse.cdt.make.core.makeBuilder"); + xml.Element("triggers", "clean,full,incremental,"); + xml.StartElement("arguments"); // use clean target - fout << - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.cleanBuildTarget\n" - "\t\t\t\t\tclean\n" - "\t\t\t\t\n" - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.enableCleanBuild\n" - "\t\t\t\t\ttrue\n" - "\t\t\t\t\n" - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.append_environment\n" - "\t\t\t\t\ttrue\n" - "\t\t\t\t\n" - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.stopOnError\n" - "\t\t\t\t\ttrue\n" - "\t\t\t\t\n" - ; + AppendDictionary(xml, "org.eclipse.cdt.make.core.cleanBuildTarget", "clean"); + AppendDictionary(xml, "org.eclipse.cdt.make.core.enableCleanBuild", "true"); + AppendDictionary(xml, "org.eclipse.cdt.make.core.append_environment","true"); + AppendDictionary(xml, "org.eclipse.cdt.make.core.stopOnError", "true"); // set the make command - std::string make = mf->GetRequiredDefinition("CMAKE_MAKE_PROGRAM"); - const std::string makeArgs = mf->GetSafeDefinition( - "CMAKE_ECLIPSE_MAKE_ARGUMENTS"); - - fout << - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.enabledIncrementalBuild\n" - "\t\t\t\t\ttrue\n" - "\t\t\t\t\n" - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.build.command\n" - "\t\t\t\t\t" << this->GetEclipsePath(make) << "\n" - "\t\t\t\t\n" - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.contents\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.activeConfigSettings\n" - "\t\t\t\t\n" - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.build.target.inc\n" - "\t\t\t\t\tall\n" - "\t\t\t\t\n" - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.build.arguments\n" - "\t\t\t\t\t" << makeArgs << "\n" - "\t\t\t\t\n" - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.buildLocation\n" - "\t\t\t\t\t" - << this->GetEclipsePath(this->HomeOutputDirectory) << "\n" - "\t\t\t\t\n" - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.useDefaultBuildCmd\n" - "\t\t\t\t\tfalse\n" - "\t\t\t\t\n" - ; + AppendDictionary(xml, "org.eclipse.cdt.make.core.enabledIncrementalBuild", + "true"); + AppendDictionary(xml, "org.eclipse.cdt.make.core.build.command", + this->GetEclipsePath(mf->GetRequiredDefinition("CMAKE_MAKE_PROGRAM"))); + AppendDictionary(xml, "org.eclipse.cdt.make.core.contents", + "org.eclipse.cdt.make.core.activeConfigSettings"); + AppendDictionary(xml, "org.eclipse.cdt.make.core.build.target.inc", "all"); + AppendDictionary(xml, "org.eclipse.cdt.make.core.build.arguments", + mf->GetSafeDefinition("CMAKE_ECLIPSE_MAKE_ARGUMENTS")); + AppendDictionary(xml, "org.eclipse.cdt.make.core.buildLocation", + this->GetEclipsePath(this->HomeOutputDirectory)); + AppendDictionary(xml, + "org.eclipse.cdt.make.core.useDefaultBuildCmd", "false"); // set project specific environment - fout << - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.environment\n" - "\t\t\t\t\tVERBOSE=1|CMAKE_NO_VERBOSE=1|" //verbose Makefile output - ; + std::stringstream environment; + environment << "VERBOSE=1|CMAKE_NO_VERBOSE=1|"; //verbose Makefile output // set vsvars32.bat environment available at CMake time, // but not necessarily when eclipse is open if (compilerId == "MSVC") { - AddEnvVar(fout, "PATH", lg); - AddEnvVar(fout, "INCLUDE", lg); - AddEnvVar(fout, "LIB", lg); - AddEnvVar(fout, "LIBPATH", lg); + AddEnvVar(environment, "PATH", lg); + AddEnvVar(environment, "INCLUDE", lg); + AddEnvVar(environment, "LIB", lg); + AddEnvVar(environment, "LIBPATH", lg); } else if (compilerId == "Intel") { // if the env.var is set, use this one and put it in the cache // if the env.var is not set, but the value is in the cache, // use it from the cache: - AddEnvVar(fout, "INTEL_LICENSE_FILE", lg); + AddEnvVar(environment, "INTEL_LICENSE_FILE", lg); } - fout << - "\n" - "\t\t\t\t\n" - ; - - fout << - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.enableFullBuild\n" - "\t\t\t\t\ttrue\n" - "\t\t\t\t\n" - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.build.target.auto\n" - "\t\t\t\t\tall\n" - "\t\t\t\t\n" - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.enableAutoBuild\n" - "\t\t\t\t\tfalse\n" - "\t\t\t\t\n" - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.build.target.clean\n" - "\t\t\t\t\tclean\n" - "\t\t\t\t\n" - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.fullBuildTarget\n" - "\t\t\t\t\tall\n" - "\t\t\t\t\n" - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.buildArguments\n" - "\t\t\t\t\t\n" - "\t\t\t\t\n" - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.build.location\n" - "\t\t\t\t\t" - << this->GetEclipsePath(this->HomeOutputDirectory) << "\n" - "\t\t\t\t\n" - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.make.core.autoBuildTarget\n" - "\t\t\t\t\tall\n" - "\t\t\t\t\n" - ; + AppendDictionary(xml, "org.eclipse.cdt.make.core.environment", + environment.str()); + + AppendDictionary(xml, "org.eclipse.cdt.make.core.enableFullBuild", "true"); + AppendDictionary(xml, "org.eclipse.cdt.make.core.build.target.auto", "all"); + AppendDictionary(xml, "org.eclipse.cdt.make.core.enableAutoBuild", "false"); + AppendDictionary(xml, "org.eclipse.cdt.make.core.build.target.clean", + "clean"); + AppendDictionary(xml, "org.eclipse.cdt.make.core.fullBuildTarget", "all"); + AppendDictionary(xml, "org.eclipse.cdt.make.core.buildArguments", ""); + AppendDictionary(xml, "org.eclipse.cdt.make.core.build.location", + this->GetEclipsePath(this->HomeOutputDirectory)); + AppendDictionary(xml, "org.eclipse.cdt.make.core.autoBuildTarget", "all"); // set error parsers - fout << - "\t\t\t\t\n" - "\t\t\t\t\torg.eclipse.cdt.core.errorOutputParser\n" - "\t\t\t\t\t" - ; + std::stringstream errorOutputParser; + if (compilerId == "MSVC") { - fout << "org.eclipse.cdt.core.VCErrorParser;"; + errorOutputParser << "org.eclipse.cdt.core.VCErrorParser;"; } else if (compilerId == "Intel") { - fout << "org.eclipse.cdt.core.ICCErrorParser;"; + errorOutputParser << "org.eclipse.cdt.core.ICCErrorParser;"; } if (this->SupportsGmakeErrorParser) { - fout << "org.eclipse.cdt.core.GmakeErrorParser;"; + errorOutputParser << "org.eclipse.cdt.core.GmakeErrorParser;"; } else { - fout << "org.eclipse.cdt.core.MakeErrorParser;"; + errorOutputParser << "org.eclipse.cdt.core.MakeErrorParser;"; } - fout << + errorOutputParser << "org.eclipse.cdt.core.GCCErrorParser;" "org.eclipse.cdt.core.GASErrorParser;" "org.eclipse.cdt.core.GLDErrorParser;" - "\n" - "\t\t\t\t\n" - ; - - fout << - "\t\t\t\n" - "\t\t\n" - "\t\t\n" - "\t\t\torg.eclipse.cdt.make.core.ScannerConfigBuilder\n" - "\t\t\t\n" - "\t\t\t\n" - "\t\t\n" - "\t\n" ; + AppendDictionary(xml, "org.eclipse.cdt.core.errorOutputParser", + errorOutputParser.str()); + + xml.EndElement(); // arguments + xml.EndElement(); // buildCommand + xml.StartElement("buildCommand"); + xml.Element("name", "org.eclipse.cdt.make.core.ScannerConfigBuilder"); + xml.StartElement("arguments"); + xml.EndElement(); // arguments + xml.EndElement(); // buildCommand + xml.EndElement(); // buildSpec // set natures for c/c++ projects - fout << - "\t\n" - "\t\torg.eclipse.cdt.make.core.makeNature\n" - "\t\torg.eclipse.cdt.make.core.ScannerConfigNature\n"; + xml.StartElement("natures"); + xml.Element("nature", "org.eclipse.cdt.make.core.makeNature"); + xml.Element("nature", "org.eclipse.cdt.make.core.ScannerConfigNature");; for (std::set::const_iterator nit=this->Natures.begin(); nit != this->Natures.end(); ++nit) { - fout << "\t\t" << *nit << "\n"; + xml.Element("nature", *nit); } if (const char *extraNaturesProp = mf->GetState() @@ -483,13 +425,13 @@ void cmExtraEclipseCDT4Generator::CreateProjectFile() for (std::vector::const_iterator nit = extraNatures.begin(); nit != extraNatures.end(); ++nit) { - fout << "\t\t" << *nit << "\n"; + xml.Element("nature", *nit); } } - fout << "\t\n"; + xml.EndElement(); // natures - fout << "\t\n"; + xml.StartElement("linkedResources"); // create linked resources if (this->IsOutOfSourceBuild) { @@ -506,7 +448,7 @@ void cmExtraEclipseCDT4Generator::CreateProjectFile() if (!cmSystemTools::IsSubDirectory(this->HomeOutputDirectory, linkSourceDirectory)) { - this->AppendLinkedResource(fout, sourceLinkedResourceName, + this->AppendLinkedResource(xml, sourceLinkedResourceName, this->GetEclipsePath(linkSourceDirectory), LinkToFolder); this->SrcLinkedResources.push_back(sourceLinkedResourceName); @@ -516,23 +458,21 @@ void cmExtraEclipseCDT4Generator::CreateProjectFile() if (this->SupportsVirtualFolders) { - this->CreateLinksToSubprojects(fout, this->HomeOutputDirectory); + this->CreateLinksToSubprojects(xml, this->HomeOutputDirectory); - this->CreateLinksForTargets(fout); + this->CreateLinksForTargets(xml); } - fout << "\t\n"; - - fout << "\n"; + xml.EndElement(); // linkedResources + xml.EndElement(); // projectDescription } //---------------------------------------------------------------------------- -void cmExtraEclipseCDT4Generator::CreateLinksForTargets( - cmGeneratedFileStream& fout) +void cmExtraEclipseCDT4Generator::CreateLinksForTargets(cmXMLWriter& xml) { std::string linkName = "[Targets]"; - this->AppendLinkedResource(fout, linkName, "virtual:/virtual",VirtualFolder); + this->AppendLinkedResource(xml, linkName, "virtual:/virtual", VirtualFolder); for (std::vector::const_iterator lgIt = this->GlobalGenerator->GetLocalGenerators().begin(); @@ -560,7 +500,7 @@ void cmExtraEclipseCDT4Generator::CreateLinksForTargets( "[exe] " : "[lib] "); linkName2 += prefix; linkName2 += (*ti)->GetName(); - this->AppendLinkedResource(fout, linkName2, "virtual:/virtual", + this->AppendLinkedResource(xml, linkName2, "virtual:/virtual", VirtualFolder); if (!this->GenerateLinkedResources) { @@ -590,7 +530,7 @@ void cmExtraEclipseCDT4Generator::CreateLinksForTargets( std::string linkName3 = linkName2; linkName3 += "/"; linkName3 += sgIt->GetFullName(); - this->AppendLinkedResource(fout, linkName3, "virtual:/virtual", + this->AppendLinkedResource(xml, linkName3, "virtual:/virtual", VirtualFolder); std::vector sFiles = sgIt->GetSourceFiles(); @@ -605,7 +545,7 @@ void cmExtraEclipseCDT4Generator::CreateLinksForTargets( std::string linkName4 = linkName3; linkName4 += "/"; linkName4 += cmSystemTools::GetFilenameName(fullPath); - this->AppendLinkedResource(fout, linkName4, + this->AppendLinkedResource(xml, linkName4, this->GetEclipsePath(fullPath), LinkToFile); } @@ -624,7 +564,7 @@ void cmExtraEclipseCDT4Generator::CreateLinksForTargets( //---------------------------------------------------------------------------- void cmExtraEclipseCDT4Generator::CreateLinksToSubprojects( - cmGeneratedFileStream& fout, const std::string& baseDir) + cmXMLWriter& xml, const std::string& baseDir) { if (!this->GenerateLinkedResources) { @@ -633,7 +573,7 @@ void cmExtraEclipseCDT4Generator::CreateLinksToSubprojects( // for each sub project create a linked resource to the source dir // - only if it is an out-of-source build - this->AppendLinkedResource(fout, "[Subprojects]", + this->AppendLinkedResource(xml, "[Subprojects]", "virtual:/virtual", VirtualFolder); for (std::map >::const_iterator @@ -651,7 +591,7 @@ void cmExtraEclipseCDT4Generator::CreateLinksToSubprojects( { std::string linkName = "[Subprojects]/"; linkName += it->first; - this->AppendLinkedResource(fout, linkName, + this->AppendLinkedResource(xml, linkName, this->GetEclipsePath(linkSourceDirectory), LinkToFolder ); @@ -664,7 +604,7 @@ void cmExtraEclipseCDT4Generator::CreateLinksToSubprojects( //---------------------------------------------------------------------------- void cmExtraEclipseCDT4Generator::AppendIncludeDirectories( - cmGeneratedFileStream& fout, + cmXMLWriter& xml, const std::vector& includeDirs, std::set& emittedDirs) { @@ -688,10 +628,13 @@ void cmExtraEclipseCDT4Generator::AppendIncludeDirectories( if(emittedDirs.find(dir) == emittedDirs.end()) { emittedDirs.insert(dir); - fout << "\n"; + xml.StartElement("pathentry"); + xml.Attribute("include", + cmExtraEclipseCDT4Generator::GetEclipsePath(dir)); + xml.Attribute("kind", "inc"); + xml.Attribute("path", ""); + xml.Attribute("system", "true"); + xml.EndElement(); } } } @@ -713,95 +656,100 @@ void cmExtraEclipseCDT4Generator::CreateCProjectFile() const return; } + cmXMLWriter xml(fout); + // add header - fout << - "\n" - "\n\n" - "\n" - "\n" - ; + xml.StartDocument("UTF-8"); + xml.ProcessingInstruction("fileVersion", "4.0.0"); + xml.StartElement("cproject"); + xml.StartElement("storageModule"); + xml.Attribute("moduleId", "org.eclipse.cdt.core.settings"); - fout << "\n"; + xml.StartElement("cconfiguration"); + xml.Attribute("id", "org.eclipse.cdt.core.default.config.1"); // Configuration settings... - fout << - "\n" - "\n" - "\n" - ; + xml.StartElement("storageModule"); + xml.Attribute("buildSystemId", + "org.eclipse.cdt.core.defaultConfigDataProvider"); + xml.Attribute("id", "org.eclipse.cdt.core.default.config.1"); + xml.Attribute("moduleId", "org.eclipse.cdt.core.settings"); + xml.Attribute("name", "Configuration"); + xml.Element("externalSettings"); + xml.StartElement("extensions"); + // TODO: refactor this out... std::string executableFormat = mf->GetSafeDefinition( "CMAKE_EXECUTABLE_FORMAT"); if (executableFormat == "ELF") { - fout << "\n" - ; - fout << "\n" - "\n" - "\n" - "\n" - ; + xml.StartElement("extension"); + xml.Attribute("id", "org.eclipse.cdt.core.ELF"); + xml.Attribute("point", "org.eclipse.cdt.core.BinaryParser"); + xml.EndElement(); // extension + + xml.StartElement("extension"); + xml.Attribute("id", "org.eclipse.cdt.core.GNU_ELF"); + xml.Attribute("point", "org.eclipse.cdt.core.BinaryParser"); + AppendAttribute(xml, "addr2line"); + AppendAttribute(xml, "c++filt"); + xml.EndElement(); // extension } else { std::string systemName = mf->GetSafeDefinition("CMAKE_SYSTEM_NAME"); if (systemName == "CYGWIN") { - fout << "\n" - "\n" - "\n" - "\n" - "\n" - "\n" - ; + xml.StartElement("extension"); + xml.Attribute("id", "org.eclipse.cdt.core.Cygwin_PE"); + xml.Attribute("point", "org.eclipse.cdt.core.BinaryParser"); + AppendAttribute(xml, "addr2line"); + AppendAttribute(xml, "c++filt"); + AppendAttribute(xml, "cygpath"); + AppendAttribute(xml, "nm"); + xml.EndElement(); // extension } else if (systemName == "Windows") { - fout << "\n" - ; + xml.StartElement("extension"); + xml.Attribute("id", "org.eclipse.cdt.core.PE"); + xml.Attribute("point", "org.eclipse.cdt.core.BinaryParser"); + xml.EndElement(); // extension } else if (systemName == "Darwin") { - fout << "SupportsMachO64Parser ? "org.eclipse.cdt.core.MachO64" - : "org.eclipse.cdt.core.MachO") << "\"" - " point=\"org.eclipse.cdt.core.BinaryParser\">\n" - "\n" - "\n" - ; + xml.StartElement("extension"); + xml.Attribute("id", this->SupportsMachO64Parser ? + "org.eclipse.cdt.core.MachO64" : "org.eclipse.cdt.core.MachO"); + xml.Attribute("point", "org.eclipse.cdt.core.BinaryParser"); + AppendAttribute(xml, "c++filt"); + xml.EndElement(); // extension } else { // *** Should never get here *** - fout << "\n"; + xml.Element("error_toolchain_type"); } } - fout << "\n" - "\n" - ; + xml.EndElement(); // extensions + xml.EndElement(); // storageModule // ??? - fout << - "\n" - "\n" - "\n" - ; + xml.StartElement("storageModule"); + xml.Attribute("moduleId", "org.eclipse.cdt.core.language.mapping"); + xml.Element("project-mappings"); + xml.EndElement(); // storageModule // ??? - fout<<"\n" - ; + xml.StartElement("storageModule"); + xml.Attribute("moduleId", "org.eclipse.cdt.core.externalSettings"); + xml.EndElement(); // storageModule // set the path entries (includes, libs, source dirs, etc.) - fout << "\n" - ; + xml.StartElement("storageModule"); + xml.Attribute("moduleId", "org.eclipse.cdt.core.pathentry"); + // for each sub project with a linked resource to the source dir: // - make it type 'src' // - and exclude it from type 'out' @@ -824,21 +772,27 @@ void cmExtraEclipseCDT4Generator::CreateCProjectFile() const it != this->SrcLinkedResources.end(); ++it) { - fout << "EscapeForXML(*it) - << "\"/>\n"; + xml.StartElement("pathentry"); + xml.Attribute("kind", "src"); + xml.Attribute("path", *it); + xml.EndElement(); // exlude source directory from output search path // - only if not named the same as an output directory if (!cmSystemTools::FileIsDirectory( std::string(this->HomeOutputDirectory + "/" + *it))) { - excludeFromOut += this->EscapeForXML(*it) + "/|"; + excludeFromOut += *it + "/|"; } } excludeFromOut += "**/CMakeFiles/"; - fout << "\n"; + + xml.StartElement("pathentry"); + xml.Attribute("excluding", excludeFromOut); + xml.Attribute("kind", "out"); + xml.Attribute("path", ""); + xml.EndElement(); // add pre-processor definitions to allow eclipse to gray out sections emmited.clear(); @@ -884,9 +838,12 @@ void cmExtraEclipseCDT4Generator::CreateCProjectFile() const if(emmited.find(def) == emmited.end()) { emmited.insert(def); - fout << "EscapeForXML(val) - << "\"/>\n"; + xml.StartElement("pathentry"); + xml.Attribute("kind", "mac"); + xml.Attribute("name", def); + xml.Attribute("path", ""); + xml.Attribute("value", val); + xml.EndElement(); } } } @@ -919,9 +876,12 @@ void cmExtraEclipseCDT4Generator::CreateCProjectFile() const if(emmited.find(def) == emmited.end()) { emmited.insert(def); - fout << "EscapeForXML(val) - << "\"/>\n"; + xml.StartElement("pathentry"); + xml.Attribute("kind", "mac"); + xml.Attribute("name", def); + xml.Attribute("path", ""); + xml.Attribute("value", val); + xml.EndElement(); } } } @@ -954,9 +914,12 @@ void cmExtraEclipseCDT4Generator::CreateCProjectFile() const if(emmited.find(def) == emmited.end()) { emmited.insert(def); - fout << "EscapeForXML(val) - << "\"/>\n"; + xml.StartElement("pathentry"); + xml.Attribute("kind", "mac"); + xml.Attribute("name", def); + xml.Attribute("path", ""); + xml.Attribute("value", val); + xml.EndElement(); } } } @@ -976,7 +939,7 @@ void cmExtraEclipseCDT4Generator::CreateCProjectFile() const std::vector includeDirs; std::string config = mf->GetSafeDefinition("CMAKE_BUILD_TYPE"); (*it)->GetIncludeDirectories(includeDirs, *l, "C", config); - this->AppendIncludeDirectories(fout, includeDirs, emmited); + this->AppendIncludeDirectories(xml, includeDirs, emmited); } } // now also the system include directories, in case we found them in @@ -989,7 +952,7 @@ void cmExtraEclipseCDT4Generator::CreateCProjectFile() const "CMAKE_EXTRA_GENERATOR_C_SYSTEM_INCLUDE_DIRS"); std::vector dirs; cmSystemTools::ExpandListArgument(systemIncludeDirs, dirs); - this->AppendIncludeDirectories(fout, dirs, emmited); + this->AppendIncludeDirectories(xml, dirs, emmited); } compiler = mf->GetSafeDefinition("CMAKE_CXX_COMPILER"); if (this->CXXEnabled && !compiler.empty()) @@ -998,16 +961,15 @@ void cmExtraEclipseCDT4Generator::CreateCProjectFile() const "CMAKE_EXTRA_GENERATOR_CXX_SYSTEM_INCLUDE_DIRS"); std::vector dirs; cmSystemTools::ExpandListArgument(systemIncludeDirs, dirs); - this->AppendIncludeDirectories(fout, dirs, emmited); + this->AppendIncludeDirectories(xml, dirs, emmited); } - fout << "\n"; + xml.EndElement(); // storageModule // add build targets - fout << - "\n" - "\n" - ; + xml.StartElement("storageModule"); + xml.Attribute("moduleId", "org.eclipse.cdt.make.core.buildtargets"); + xml.StartElement("buildTargets"); emmited.clear(); const std::string make = mf->GetRequiredDefinition("CMAKE_MAKE_PROGRAM"); const std::string makeArgs = mf->GetSafeDefinition( @@ -1055,7 +1017,7 @@ void cmExtraEclipseCDT4Generator::CreateCProjectFile() const // not from the subdirs if (subdir.empty()) { - this->AppendTarget(fout, targetName, make, makeArgs, subdir, ": "); + this->AppendTarget(xml, targetName, make, makeArgs, subdir, ": "); } } break; @@ -1070,7 +1032,7 @@ void cmExtraEclipseCDT4Generator::CreateCProjectFile() const break; } - this->AppendTarget(fout, targetName, make, makeArgs, subdir, ": "); + this->AppendTarget(xml, targetName, make, makeArgs, subdir, ": "); break; case cmState::EXECUTABLE: case cmState::STATIC_LIBRARY: @@ -1080,10 +1042,10 @@ void cmExtraEclipseCDT4Generator::CreateCProjectFile() const { const char* prefix = ((*ti)->GetType()==cmState::EXECUTABLE ? "[exe] " : "[lib] "); - this->AppendTarget(fout, targetName, make, makeArgs, subdir, prefix); + this->AppendTarget(xml, targetName, make, makeArgs, subdir, prefix); std::string fastTarget = targetName; fastTarget += "/fast"; - this->AppendTarget(fout, fastTarget, make, makeArgs, subdir, prefix); + this->AppendTarget(xml, fastTarget, make, makeArgs, subdir, prefix); // Add Build and Clean targets in the virtual folder of targets: if (this->SupportsVirtualFolders) @@ -1095,7 +1057,7 @@ void cmExtraEclipseCDT4Generator::CreateCProjectFile() const buildArgs += (*it)->GetBinaryDirectory(); buildArgs += "\" "; buildArgs += makeArgs; - this->AppendTarget(fout, "Build", make, buildArgs, virtDir, "", + this->AppendTarget(xml, "Build", make, buildArgs, virtDir, "", targetName.c_str()); std::string cleanArgs = "-E chdir \""; @@ -1106,7 +1068,7 @@ void cmExtraEclipseCDT4Generator::CreateCProjectFile() const cmGeneratorTarget* gt = *ti; cleanArgs += (*it)->GetTargetDirectory(gt); cleanArgs += "/cmake_clean.cmake\""; - this->AppendTarget(fout, "Clean", cmSystemTools::GetCMakeCommand(), + this->AppendTarget(xml, "Clean", cmSystemTools::GetCMakeCommand(), cleanArgs, virtDir, "", ""); } } @@ -1119,11 +1081,11 @@ void cmExtraEclipseCDT4Generator::CreateCProjectFile() const // insert the all and clean targets in every subdir if (!allTarget.empty()) { - this->AppendTarget(fout, allTarget, make, makeArgs, subdir, ": "); + this->AppendTarget(xml, allTarget, make, makeArgs, subdir, ": "); } if (!cleanTarget.empty()) { - this->AppendTarget(fout, cleanTarget, make, makeArgs, subdir, ": "); + this->AppendTarget(xml, cleanTarget, make, makeArgs, subdir, ": "); } //insert rules for compiling, preprocessing and assembling individual files @@ -1142,25 +1104,29 @@ void cmExtraEclipseCDT4Generator::CreateCProjectFile() const { prefix = "[pre] "; } - this->AppendTarget(fout, *fit, make, makeArgs, subdir, prefix); + this->AppendTarget(xml, *fit, make, makeArgs, subdir, prefix); } } - fout << "\n" - "\n" - ; - - this->AppendStorageScanners(fout, *mf); - - fout << "\n" - "\n" - "\n" - "EscapeForXML(lg->GetProjectName()) - << ".null.1\" name=\"" << this->EscapeForXML(lg->GetProjectName()) - << "\"/>\n" - "\n" - "\n" - ; + xml.EndElement(); // buildTargets + xml.EndElement(); // storageModule + + this->AppendStorageScanners(xml, *mf); + + xml.EndElement(); // cconfiguration + xml.EndElement(); // storageModule + + xml.StartElement("storageModule"); + xml.Attribute("moduleId", "cdtBuildSystem"); + xml.Attribute("version", "4.0.0"); + + xml.StartElement("project"); + xml.Attribute("id", std::string(lg->GetProjectName()) + ".null.1"); + xml.Attribute("name", lg->GetProjectName()); + xml.EndElement(); // project + + xml.EndElement(); // storageModule + xml.EndElement(); // cproject } //---------------------------------------------------------------------------- @@ -1208,28 +1174,14 @@ cmExtraEclipseCDT4Generator::GenerateProjectName(const std::string& name, const std::string& type, const std::string& path) { - return cmExtraEclipseCDT4Generator::EscapeForXML(name) - +(type.empty() ? "" : "-") + type + "@" + path; -} - -std::string cmExtraEclipseCDT4Generator::EscapeForXML(const std::string& value) -{ - std::string str = value; - cmSystemTools::ReplaceString(str, "&", "&"); - cmSystemTools::ReplaceString(str, "<", "<"); - cmSystemTools::ReplaceString(str, ">", ">"); - cmSystemTools::ReplaceString(str, "\"", """); - // NOTE: This one is not necessary, since as of Eclipse CDT4 it will - // automatically change this to the original value ('). - //cmSystemTools::ReplaceString(str, "'", "'"); - return str; + return name + (type.empty() ? "" : "-") + type + "@" + path; } //---------------------------------------------------------------------------- // Helper functions //---------------------------------------------------------------------------- void cmExtraEclipseCDT4Generator -::AppendStorageScanners(cmGeneratedFileStream& fout, +::AppendStorageScanners(cmXMLWriter& xml, const cmMakefile& makefile) { // we need the "make" and the C (or C++) compiler which are used, Alex @@ -1255,24 +1207,28 @@ void cmExtraEclipseCDT4Generator compilerArgs = arg1 + compilerArgs; } - fout << - "\n" - "\n" - ; - cmExtraEclipseCDT4Generator::AppendScannerProfile(fout, + xml.StartElement("storageModule"); + xml.Attribute("moduleId", "scannerConfiguration"); + + xml.StartElement("autodiscovery"); + xml.Attribute("enabled", "true"); + xml.Attribute("problemReportingEnabled", "true"); + xml.Attribute("selectedProfileId", + "org.eclipse.cdt.make.core.GCCStandardMakePerProjectProfile"); + xml.EndElement(); // autodiscovery + + cmExtraEclipseCDT4Generator::AppendScannerProfile(xml, "org.eclipse.cdt.make.core.GCCStandardMakePerProjectProfile", true, "", true, "specsFile", compilerArgs, compiler, true, true); - cmExtraEclipseCDT4Generator::AppendScannerProfile(fout, + cmExtraEclipseCDT4Generator::AppendScannerProfile(xml, "org.eclipse.cdt.make.core.GCCStandardMakePerFileProfile", true, "", true, "makefileGenerator", "-f ${project_name}_scd.mk", make, true, true); - fout << "\n"; + xml.EndElement(); // storageModule } // The prefix is prepended before the actual name of the target. The purpose @@ -1283,7 +1239,7 @@ void cmExtraEclipseCDT4Generator // finally the assembly files "[to asm] ". Note the "to" in "to asm", // without it, "asm" would be the first targets in the list, with the "to" // they are the last targets, which makes more sense. -void cmExtraEclipseCDT4Generator::AppendTarget(cmGeneratedFileStream& fout, +void cmExtraEclipseCDT4Generator::AppendTarget(cmXMLWriter& xml, const std::string& target, const std::string& make, const std::string& makeArgs, @@ -1292,31 +1248,21 @@ void cmExtraEclipseCDT4Generator::AppendTarget(cmGeneratedFileStream& fout, const char* makeTarget ) { - std::string targetXml = cmExtraEclipseCDT4Generator::EscapeForXML(target); - std::string makeTargetXml = targetXml; - if (makeTarget != NULL) - { - makeTargetXml = cmExtraEclipseCDT4Generator::EscapeForXML(makeTarget); - } - cmExtraEclipseCDT4Generator::EscapeForXML(target); - std::string pathXml = cmExtraEclipseCDT4Generator::EscapeForXML(path); - fout << - "\n" - "" - << cmExtraEclipseCDT4Generator::GetEclipsePath(make) - << "\n" - "" << makeArgs << "\n" - "" << makeTargetXml << "\n" - "true\n" - "false\n" - "\n" - ; + xml.StartElement("target"); + xml.Attribute("name", prefix + target); + xml.Attribute("path", path); + xml.Attribute("targetID", "org.eclipse.cdt.make.MakeTargetBuilder"); + xml.Element("buildCommand", + cmExtraEclipseCDT4Generator::GetEclipsePath(make)); + xml.Element("buildArguments", makeArgs); + xml.Element("buildTarget", makeTarget ? makeTarget : target.c_str()); + xml.Element("stopOnError", "true"); + xml.Element("useDefaultCommand", "false"); + xml.EndElement(); } void cmExtraEclipseCDT4Generator -::AppendScannerProfile(cmGeneratedFileStream& fout, +::AppendScannerProfile(cmXMLWriter& xml, const std::string& profileID, bool openActionEnabled, const std::string& openActionFilePath, @@ -1327,49 +1273,54 @@ void cmExtraEclipseCDT4Generator bool runActionUseDefault, bool sipParserEnabled) { - fout << - "\n" - "\n" - "\n" - "\n" - "\n" - "\n" - "\n" - "\n" - "\n" - "\n" - ; + xml.StartElement("profile"); + xml.Attribute("id", profileID); + + xml.StartElement("buildOutputProvider"); + xml.StartElement("openAction"); + xml.Attribute("enabled", openActionEnabled ? "true" : "false"); + xml.Attribute("filePath", openActionFilePath); + xml.EndElement(); // openAction + xml.StartElement("parser"); + xml.Attribute("enabled", pParserEnabled ? "true" : "false"); + xml.EndElement(); // parser + xml.EndElement(); // buildOutputProvider + + xml.StartElement("scannerInfoProvider"); + xml.Attribute("id", scannerInfoProviderID); + xml.StartElement("runAction"); + xml.Attribute("arguments", runActionArguments); + xml.Attribute("command", runActionCommand); + xml.Attribute("useDefault", runActionUseDefault ? "true" : "false"); + xml.EndElement(); // runAction + xml.StartElement("parser"); + xml.Attribute("enabled", sipParserEnabled ? "true" : "false"); + xml.EndElement(); // parser + xml.EndElement(); // scannerInfoProvider + + xml.EndElement(); // profile } void cmExtraEclipseCDT4Generator -::AppendLinkedResource (cmGeneratedFileStream& fout, +::AppendLinkedResource (cmXMLWriter& xml, const std::string& name, const std::string& path, LinkType linkType) { const char* locationTag = "location"; - const char* typeTag = "2"; + int typeTag = 2; if (linkType == VirtualFolder) // ... and not a linked folder { locationTag = "locationURI"; } if (linkType == LinkToFile) { - typeTag = "1"; + typeTag = 1; } - fout << - "\t\t\n" - "\t\t\t" - << cmExtraEclipseCDT4Generator::EscapeForXML(name) - << "\n" - "\t\t\t" << typeTag << "\n" - "\t\t\t<" << locationTag << ">" - << cmExtraEclipseCDT4Generator::EscapeForXML(path) - << "\n" - "\t\t\n" - ; + xml.StartElement("link"); + xml.Element("name", name); + xml.Element("type", typeTag); + xml.Element(locationTag, path); + xml.EndElement(); } diff --git a/Source/cmExtraEclipseCDT4Generator.h b/Source/cmExtraEclipseCDT4Generator.h index 1da2077..4032a6c 100644 --- a/Source/cmExtraEclipseCDT4Generator.h +++ b/Source/cmExtraEclipseCDT4Generator.h @@ -17,7 +17,7 @@ #include "cmExternalMakefileProjectGenerator.h" class cmMakefile; -class cmGeneratedFileStream; +class cmXMLWriter; /** \class cmExtraEclipseCDT4Generator * \brief Write Eclipse project files for Makefile based projects @@ -67,19 +67,17 @@ private: const std::string& type, const std::string& path); - static std::string EscapeForXML(const std::string& value); - // Helper functions - static void AppendStorageScanners(cmGeneratedFileStream& fout, + static void AppendStorageScanners(cmXMLWriter& xml, const cmMakefile& makefile); - static void AppendTarget (cmGeneratedFileStream& fout, + static void AppendTarget (cmXMLWriter& xml, const std::string& target, const std::string& make, const std::string& makeArguments, const std::string& path, const char* prefix = "", const char* makeTarget = NULL); - static void AppendScannerProfile (cmGeneratedFileStream& fout, + static void AppendScannerProfile (cmXMLWriter& xml, const std::string& profileID, bool openActionEnabled, const std::string& openActionFilePath, @@ -90,21 +88,20 @@ private: bool runActionUseDefault, bool sipParserEnabled); - static void AppendLinkedResource (cmGeneratedFileStream& fout, + static void AppendLinkedResource (cmXMLWriter& xml, const std::string& name, const std::string& path, LinkType linkType); - static void AppendIncludeDirectories(cmGeneratedFileStream& fout, + static void AppendIncludeDirectories(cmXMLWriter& xml, const std::vector& includeDirs, std::set& emittedDirs); - static void AddEnvVar(cmGeneratedFileStream& fout, const char* envVar, + static void AddEnvVar(std::ostream& out, const char* envVar, cmLocalGenerator* lg); - void CreateLinksToSubprojects(cmGeneratedFileStream& fout, - const std::string& baseDir); - void CreateLinksForTargets(cmGeneratedFileStream& fout); + void CreateLinksToSubprojects(cmXMLWriter& xml, const std::string& baseDir); + void CreateLinksForTargets(cmXMLWriter& xml); std::vector SrcLinkedResources; std::set Natures; -- cgit v0.12