Browse Source

BUG: Dependency hints must be removed by the makefile generator before adding the dependencies generated by cmMakeDepend.

Brad King 25 years ago
parent
commit
5f12ae0c62
2 changed files with 5 additions and 14 deletions
  1. 0 14
      Source/cmMakeDepend.cxx
  2. 5 0
      Source/cmUnixMakefileGenerator.cxx

+ 0 - 14
Source/cmMakeDepend.cxx

@@ -169,15 +169,6 @@ void cmMakeDepend::Depend(cmDependInformation* info)
   // If the file exists, use it to find dependency information.
   if(cmSystemTools::FileExists(path))
     {
-    // The cmSourceFile may have had hints for dependencies.  Delete any that
-    // exist since we can find the dependencies for real.
-    if(info->m_ClassFileIndex != 0)
-      {
-      const cmSourceFile& cFile = *(info->m_ClassFileIndex);
-      //cFile.GetDepends().erase(cFile.GetDepends().begin(), 
-      //                         cFile.GetDepends().end());
-      }
-    
     // Use the real file to find its dependencies.
     this->DependWalk(info, path);
     info->m_DependDone = true;
@@ -203,11 +194,6 @@ void cmMakeDepend::Depend(cmDependInformation* info)
         this->AddDependency(info, file->c_str());
         }
       
-      // Erase the dependency hints from the cmSourceFile.  They will be
-      // put in again as real dependencies later.
-      //cFile.GetDepends().erase(cFile.GetDepends().begin(), 
-      //                         cFile.GetDepends().end());
-      
       // Found dependency information.  We are done.
       return;
       }

+ 5 - 0
Source/cmUnixMakefileGenerator.cxx

@@ -93,6 +93,11 @@ void cmUnixMakefileGenerator::ProcessDepends(const cmMakeDepend &md)
         // get the depends
         const cmDependInformation *info = 
           md.GetDependInformationForSourceFile(*i);
+        
+        // Delete any hints from the source file's dependencies.
+        i->GetDepends().erase(i->GetDepends().begin(), i->GetDepends().end());
+        
+        // Now add the real dependencies for the file.
         if (info)
           {
           for( cmDependInformation::IndexSet::const_iterator indx =