Browse Source

BUG: handle dependencies to imported targets better: don't create a
dependency if the target name was not listed in DEPENDS, if it was listed in
DEPENDS, create a dependency to the file

Seems to work, but have to check with Brad.

Alex

Alexander Neundorf 18 years ago
parent
commit
e126db1f71
2 changed files with 3 additions and 3 deletions
  1. 1 1
      Source/cmLocalGenerator.cxx
  2. 2 2
      Source/cmTarget.cxx

+ 1 - 1
Source/cmLocalGenerator.cxx

@@ -1964,7 +1964,7 @@ std::string cmLocalGenerator::GetRealDependency(const char* inName,
     }
 
   // Look for a CMake target with the given name.
-  if(cmTarget* target = this->GlobalGenerator->FindTarget(0,name.c_str(),false))
+  if(cmTarget* target = this->GlobalGenerator->FindTarget(0,name.c_str(),true))
     {
     // make sure it is not just a coincidence that the target name
     // found is part of the inName

+ 2 - 2
Source/cmTarget.cxx

@@ -538,9 +538,9 @@ bool cmTargetTraceDependencies::IsUtility(std::string const& dep)
     util = cmSystemTools::GetFilenameWithoutLastExtension(util);
     }
 
-  // Check for a non-imported target with this name.
+  // Check for a target with this name.
   if(cmTarget* t =
-     this->GlobalGenerator->FindTarget(0, util.c_str(), false))
+     this->GlobalGenerator->FindTarget(0, util.c_str(), true))
     {
     // If we find the target and the dep was given as a full path,
     // then make sure it was not a full path to something else, and