ENH: Fixed INSTALL_FILES and INSTALL_PROGRAMS commands to not create targets. No targets of type cmTarget::INSTALL_FILES or cmTarget::INSTALL_PROGRAMS are created, so we do not need to check for them everywhere anymore.

1 files changed, 15 insertions, 19 deletions

diff --git a/Source/cmExportLibraryDependencies.cxx b/Source/cmExportLibraryDependencies.cxx
index a5d66e75b2..bf9d5339b2 100644
--- a/Source/cmExportLibraryDependencies.cxx
+++ b/Source/cmExportLibraryDependencies.cxx
@@ -94,28 +94,24 @@ void cmExportLibraryDependenciesCommand::FinalPass()
     for(cmTargets::iterator l = tgts.begin();
         l != tgts.end(); ++l)
       {
-      if ((l->second.GetType() != cmTarget::INSTALL_FILES)
-          && (l->second.GetType() != cmTarget::INSTALL_PROGRAMS))
+      libDepName = l->first;
+      libDepName += "_LIB_DEPENDS";
+      const char* def = this->Makefile->GetDefinition(libDepName.c_str());
+      if(def)
         {
-        libDepName = l->first;
-        libDepName += "_LIB_DEPENDS";
-        const char* def = this->Makefile->GetDefinition(libDepName.c_str());
-        if(def)
+        fout << "SET(" << libDepName << " \"" << def << "\")\n";
+        // now for each dependency, check for link type
+        cmSystemTools::ExpandListArgument(def, depends);
+        for(std::vector<std::string>::const_iterator d = depends.begin();
+            d != depends.end(); ++d)
           {
-          fout << "SET(" << libDepName << " \"" << def << "\")\n";
-          // now for each dependency, check for link type
-          cmSystemTools::ExpandListArgument(def, depends);
-          for(std::vector<std::string>::const_iterator d = depends.begin();
-              d != depends.end(); ++d)
+          libDepName = *d;
+          libDepName += "_LINK_TYPE";
+          defType = this->Makefile->GetDefinition(libDepName.c_str());
+          libDepName = cmSystemTools::EscapeSpaces(libDepName.c_str());
+          if(defType)
             {
-            libDepName = *d;
-            libDepName += "_LINK_TYPE";
-            defType = this->Makefile->GetDefinition(libDepName.c_str());
-            libDepName = cmSystemTools::EscapeSpaces(libDepName.c_str());
-            if(defType)
-              {
-              fout << "SET(" << libDepName << " \"" << defType << "\")\n";
-              }
+            fout << "SET(" << libDepName << " \"" << defType << "\")\n";
             }
           }
         }