[Cmake-commits] [cmake-commits] king committed cmComputeLinkDepends.cxx 1.14 1.15

[cmake-commits at cmake.org]

Update of /cvsroot/CMake/CMake/Source
In directory public:/mounts/ram/cvs-serv18222/Source

Modified Files:
	cmComputeLinkDepends.cxx 
Log Message:
ENH: Simplify link lines in some cases by not allowing targets to be inferred dependees of items with unknown dependencies.


Index: cmComputeLinkDepends.cxx
===================================================================
RCS file: /cvsroot/CMake/CMake/Source/cmComputeLinkDepends.cxx,v
retrieving revision 1.14
retrieving revision 1.15
diff -C 2 -d -r1.14 -r1.15
*** cmComputeLinkDepends.cxx	31 Mar 2008 16:47:31 -0000	1.14
--- cmComputeLinkDepends.cxx	23 Apr 2008 04:40:12 -0000	1.15
***************
*** 101,104 ****
--- 101,107 ----
    C: intersect( {}      , {B}   ) = {}    ; infer no edges
  
+ Note that targets are never inferred as dependees because outside
+ libraries should not depend on them.
+ 
  ------------------------------------------------------------------------------
  
***************
*** 549,553 ****
          dsi != dependSets.end(); ++dsi)
        {
!       if(dependee_index != dsi->first)
          {
          dsi->second.insert(dependee_index);
--- 552,561 ----
          dsi != dependSets.end(); ++dsi)
        {
!       // Add this item to the inferred dependencies of other items.
!       // Target items are never inferred dependees because unknown
!       // items are outside libraries that should not be depending on
!       // targets.
!       if(!this->EntryList[dependee_index].Target &&
!          dependee_index != dsi->first)
          {
          dsi->second.insert(dependee_index);