From a2fadc80fa4f746327ff4908075335ebc8d4a8b0 Mon Sep 17 00:00:00 2001 From: Alexander Neundorf Date: Tue, 19 Feb 2008 16:27:03 -0500 Subject: [PATCH] ENH: patch from Miguel part 2: if ECLIPSE_CDT4_GENERATE_SOURCE_PROJECT is true, then the generator additionally generates eclipse project files in the source dir, since this is the only way to get cvs/svn working with eclipse This is off by default and the user has to enable it explicitely. If cmake can't write there it still continues. Alex --- Source/cmExtraEclipseCDT4Generator.cxx | 156 ++++++++++++++----------- Source/cmExtraEclipseCDT4Generator.h | 13 ++- 2 files changed, 103 insertions(+), 66 deletions(-) diff --git a/Source/cmExtraEclipseCDT4Generator.cxx b/Source/cmExtraEclipseCDT4Generator.cxx index affc0fda5..b7461ebfd 100644 --- a/Source/cmExtraEclipseCDT4Generator.cxx +++ b/Source/cmExtraEclipseCDT4Generator.cxx @@ -26,6 +26,7 @@ #include "cmSystemTools.h" #include +#include //---------------------------------------------------------------------------- cmExtraEclipseCDT4Generator @@ -78,6 +79,20 @@ void cmExtraEclipseCDT4Generator::Generate() this->HomeDirectory = mf->GetHomeDirectory(); this->HomeOutputDirectory = mf->GetHomeOutputDirectory(); + this->IsOutOfSourceBuild = (this->HomeDirectory!=this->HomeOutputDirectory); + + this->GenerateSourceProject = (this->IsOutOfSourceBuild && + mf->IsOn("ECLIPSE_CDT4_GENERATE_SOURCE_PROJECT")); + + // NOTE: This is not good, since it pollutes the source tree. However, + // Eclipse doesn't allow CVS/SVN to work when the .project is not in + // the cvs/svn root directory. Hence, this is provided as an option. + if (this->GenerateSourceProject) + { + // create .project file in the source tree + this->CreateSourceProjectFile(); + } + // create a .project file this->CreateProjectFile(); @@ -85,14 +100,43 @@ void cmExtraEclipseCDT4Generator::Generate() this->CreateCProjectFile(); } +void cmExtraEclipseCDT4Generator::CreateSourceProjectFile() const +{ + assert(this->HomeDirectory != this->HomeOutputDirectory); + + // set up the project name: -Source@ + const cmMakefile* mf + = this->GlobalGenerator->GetLocalGenerators()[0]->GetMakefile(); + std::string name = this->GenerateProjectName(mf->GetProjectName(), "Source", + this->GetPathBasename(this->HomeDirectory)); + + const std::string filename = this->HomeDirectory + "/.project"; + cmGeneratedFileStream fout(filename.c_str()); + if (!fout) + { + return; + } + + fout << + "\n" + "\n" + "\t" << name << "\n" + "\t\n" + "\t\n" + "\t\n" + "\t\n" + "\t\n" + "\t\n" + "\t\n" + "\n" + ; +} + //---------------------------------------------------------------------------- -void cmExtraEclipseCDT4Generator::CreateProjectFile() const +void cmExtraEclipseCDT4Generator::CreateProjectFile() { const cmMakefile* mf = this->GlobalGenerator->GetLocalGenerators()[0]->GetMakefile(); - // set up the project name: -Source@ - std::string name = this->GenerateProjectName(mf->GetProjectName(), "Source", - this->GetPathBasename(this->HomeDirectory)); const std::string filename = this->HomeOutputDirectory + "/.project"; @@ -285,7 +329,7 @@ void cmExtraEclipseCDT4Generator::CreateProjectFile() const // TODO: refactor this // create linked resources - if (this->HomeDirectory != this->HomeOutputDirectory) + if (this->IsOutOfSourceBuild) { fout << "\t\n"; // for each sub project create a linked resource to the source dir @@ -297,18 +341,13 @@ void cmExtraEclipseCDT4Generator::CreateProjectFile() const { std::string linkSourceDirectory =this->GetEclipsePath( it->second[0]->GetMakefile()->GetStartDirectory()); + // .project dir can't be subdir of a linked resource dir if (!cmSystemTools::IsSubDirectory(this->HomeOutputDirectory.c_str(), linkSourceDirectory.c_str())) { - fout << - "\t\t\n" - "\t\t\t" << it->first << "\n" - "\t\t\t2\n" - "\t\t\t" - << this->GetEclipsePath(linkSourceDirectory) - << "\n" - "\t\t\n" - ; + this->AppendLinkedResource(fout, it->first, + this->GetEclipsePath(linkSourceDirectory)); + this->SrcLinkedResources.push_back(it->first); } } // for EXECUTABLE_OUTPUT_PATH when not in binary dir @@ -317,20 +356,16 @@ void cmExtraEclipseCDT4Generator::CreateProjectFile() const this->HomeOutputDirectory.c_str())) { std::string name = this->GetPathBasename(output_path); + + // make sure linked resource name is unique while (this->GlobalGenerator->GetProjectMap().find(name) != this->GlobalGenerator->GetProjectMap().end()) { name += "_"; } - fout << - "\t\t\n" - "\t\t\t" << name << "\n" - "\t\t\t2\n" - "\t\t\t" - << this->GetEclipsePath(output_path) - << "\n" - "\t\t\n" - ; + this->AppendLinkedResource(fout, name, + this->GetEclipsePath(output_path)); + this->OutLinkedResources.push_back(name); } // for LIBRARY_OUTPUT_PATH when not in binary dir if (output_path != mf->GetDefinition("LIBRARY_OUTPUT_PATH")) @@ -340,20 +375,16 @@ void cmExtraEclipseCDT4Generator::CreateProjectFile() const this->HomeOutputDirectory.c_str())) { std::string name = this->GetPathBasename(output_path); + + // make sure linked resource name is unique while (this->GlobalGenerator->GetProjectMap().find(name) != this->GlobalGenerator->GetProjectMap().end()) { name += "_"; } - fout << - "\t\t\n" - "\t\t\t" << name << "\n" - "\t\t\t2\n" - "\t\t\t" - << this->GetEclipsePath(output_path) - << "\n" - "\t\t\n" - ; + this->AppendLinkedResource(fout, name, + this->GetEclipsePath(output_path)); + this->OutLinkedResources.push_back(name); } } fout << "\t\n"; @@ -472,19 +503,19 @@ void cmExtraEclipseCDT4Generator::CreateCProjectFile() const // - make it type 'src' // - and exclude it from type 'out' std::string exclude_from_out; - for (std::map >::const_iterator - it = this->GlobalGenerator->GetProjectMap().begin(); - it != this->GlobalGenerator->GetProjectMap().end(); + for (std::vector::const_iterator + it = this->SrcLinkedResources.begin(); + it != this->SrcLinkedResources.end(); ++it) { - fout << "first << "\"/>\n"; + fout << "\n"; // 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->first).c_str())) + std::string(this->HomeOutputDirectory + "/" + *it).c_str())) { - exclude_from_out += it->first + "/|"; + exclude_from_out += *it + "/|"; } } exclude_from_out += "**/CMakeFiles/"; @@ -494,33 +525,12 @@ void cmExtraEclipseCDT4Generator::CreateCProjectFile() const // - if it is a subdir of homeOutputDirectory, there is no need to add it // - if it is not then create a linked resource and add the linked name // but check it doesn't conflict with other linked resources names - std::string output_path = mf->GetDefinition("EXECUTABLE_OUTPUT_PATH"); - if (!cmSystemTools::IsSubDirectory(output_path.c_str(), - this->HomeOutputDirectory.c_str())) + for (std::vector::const_iterator + it = this->OutLinkedResources.begin(); + it != this->OutLinkedResources.end(); + ++it) { - std::string name = this->GetPathBasename(output_path); - while (this->GlobalGenerator->GetProjectMap().find(name) - != this->GlobalGenerator->GetProjectMap().end()) - { - name += "_"; - } - fout << "\n"; - } - // for LIBRARY_OUTPUT_PATH when not in binary dir - if (output_path != mf->GetDefinition("LIBRARY_OUTPUT_PATH")) - { - output_path = mf->GetDefinition("LIBRARY_OUTPUT_PATH"); - if (!cmSystemTools::IsSubDirectory(output_path.c_str(), - this->HomeOutputDirectory.c_str())) - { - std::string name = this->GetPathBasename(output_path); - while (this->GlobalGenerator->GetProjectMap().find(name) - != this->GlobalGenerator->GetProjectMap().end()) - { - name += "_"; - } - fout << "\n"; - } + fout << "\n"; } // add pre-processor definitions to allow eclipse to gray out sections @@ -543,7 +553,7 @@ void cmExtraEclipseCDT4Generator::CreateCProjectFile() const { std::string::size_type equals = di->find('=', 0); std::string::size_type enddef = di->length(); - + std::string def; std::string val; if (equals != std::string::npos && equals < enddef) @@ -843,3 +853,19 @@ void cmExtraEclipseCDT4Generator "\n" ; } + +void cmExtraEclipseCDT4Generator +::AppendLinkedResource (cmGeneratedFileStream& fout, + const std::string& name, + const std::string& path) +{ + fout << + "\t\t\n" + "\t\t\t" << name << "\n" + "\t\t\t2\n" + "\t\t\t" + << path + << "\n" + "\t\t\n" + ; +} diff --git a/Source/cmExtraEclipseCDT4Generator.h b/Source/cmExtraEclipseCDT4Generator.h index d025c89ce..aa9da23e1 100644 --- a/Source/cmExtraEclipseCDT4Generator.h +++ b/Source/cmExtraEclipseCDT4Generator.h @@ -52,8 +52,11 @@ public: virtual void Generate(); private: + // create .project file in the source tree + void CreateSourceProjectFile() const; + // create .project file - void CreateProjectFile() const; + void CreateProjectFile(); // create .cproject file void CreateCProjectFile() const; @@ -97,8 +100,16 @@ private: bool runActionUseDefault, bool sipParserEnabled); + static void AppendLinkedResource (cmGeneratedFileStream& fout, + const std::string& name, + const std::string& path); + + std::vector SrcLinkedResources; + std::vector OutLinkedResources; std::string HomeDirectory; std::string HomeOutputDirectory; + bool IsOutOfSourceBuild; + bool GenerateSourceProject; };