summaryrefslogtreecommitdiffstats
path: root/Source/cmGlobalVisualStudio7Generator.cxx
diff options
context:
space:
mode:
authorBrad King <brad.king@kitware.com>2009-10-06 17:30:00 (GMT)
committerBrad King <brad.king@kitware.com>2009-10-06 17:30:00 (GMT)
commit9000b5a4ded0d805ea1cd7b82f9e87e9abcf6c31 (patch)
treeaef1cee06fabc865ba60710c52edbcddd1e7463b /Source/cmGlobalVisualStudio7Generator.cxx
parentd8efcfc787b08e52c28611347e4c60cddde00aa2 (diff)
downloadCMake-9000b5a4ded0d805ea1cd7b82f9e87e9abcf6c31.zip
CMake-9000b5a4ded0d805ea1cd7b82f9e87e9abcf6c31.tar.gz
CMake-9000b5a4ded0d805ea1cd7b82f9e87e9abcf6c31.tar.bz2
Avoid non-root copies of root-only targets
In cmGlobalGenerator::GetTargetSets we collect targets from all local generators in a tree or subtree corresponding to a project() command. Some targets, such as ALL_BUILD, are duplicated in each subdirectory with a project() command. For such targets we should keep only the copy for the top-most (root) local generator. Previously this filtering was done in each VS IDE generator, but it is easier to do it in one place when the targets are first encountered. This also fixes bad ALL_BUILD dependencies generated for VS 7.0 because the cmGlobalVisualStudio7Generator::WriteTargetDepends method was not filtering out duplicates. Now we avoid duplicates from the start.
Diffstat (limited to 'Source/cmGlobalVisualStudio7Generator.cxx')
-rw-r--r--Source/cmGlobalVisualStudio7Generator.cxx43
1 files changed, 16 insertions, 27 deletions
diff --git a/Source/cmGlobalVisualStudio7Generator.cxx b/Source/cmGlobalVisualStudio7Generator.cxx
index f2f3e98..dbe9aed 100644
--- a/Source/cmGlobalVisualStudio7Generator.cxx
+++ b/Source/cmGlobalVisualStudio7Generator.cxx
@@ -289,34 +289,16 @@ void cmGlobalVisualStudio7Generator::WriteTargetsToSolution(
}
else
{
- bool skip = false;
- // if it is a global target or the check build system target
- // or the all_build target
- // then only use the one that is for the root
- if(target->GetType() == cmTarget::GLOBAL_TARGET
- || !strcmp(target->GetName(), CMAKE_CHECK_BUILD_SYSTEM_TARGET)
- || !strcmp(target->GetName(), this->GetAllTargetName()))
+ const char *vcprojName =
+ target->GetProperty("GENERATOR_FILE_NAME");
+ if(vcprojName)
{
- if(target->GetMakefile() != root->GetMakefile())
- {
- skip = true;
- }
- }
- // if not skipping the project then write it into the
- // solution
- if(!skip)
- {
- const char *vcprojName =
- target->GetProperty("GENERATOR_FILE_NAME");
- if(vcprojName)
- {
- cmMakefile* tmf = target->GetMakefile();
- std::string dir = tmf->GetStartOutputDirectory();
- dir = root->Convert(dir.c_str(),
- cmLocalGenerator::START_OUTPUT);
- this->WriteProject(fout, vcprojName, dir.c_str(),
- *target);
- }
+ cmMakefile* tmf = target->GetMakefile();
+ std::string dir = tmf->GetStartOutputDirectory();
+ dir = root->Convert(dir.c_str(),
+ cmLocalGenerator::START_OUTPUT);
+ this->WriteProject(fout, vcprojName, dir.c_str(),
+ *target);
}
}
}
@@ -633,6 +615,13 @@ cmGlobalVisualStudio7Generator
}
}
+//----------------------------------------------------------------------------
+bool cmGlobalVisualStudio7Generator::IsRootOnlyTarget(cmTarget* target)
+{
+ return (this->cmGlobalVisualStudioGenerator::IsRootOnlyTarget(target) ||
+ strcmp(target->GetName(), CMAKE_CHECK_BUILD_SYSTEM_TARGET) == 0);
+}
+
bool cmGlobalVisualStudio7Generator::IsPartOfDefaultBuild(const char* project,
cmTarget* target)
{