You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ar...@apache.org on 2011/07/19 00:48:49 UTC

svn commit: r1148083 - /subversion/trunk/build/generator/gen_make.py

Author: arfrever
Date: Mon Jul 18 22:48:46 2011
New Revision: 1148083

URL: http://svn.apache.org/viewvc?rev=1148083&view=rev
Log:
* build/generator/gen_make.py
  (Generator._get_all_lib_deps): Use a set instead of a dictionary with True
   values.

Modified:
    subversion/trunk/build/generator/gen_make.py

Modified: subversion/trunk/build/generator/gen_make.py
URL: http://svn.apache.org/viewvc/subversion/trunk/build/generator/gen_make.py?rev=1148083&r1=1148082&r2=1148083&view=diff
==============================================================================
--- subversion/trunk/build/generator/gen_make.py (original)
+++ subversion/trunk/build/generator/gen_make.py Mon Jul 18 22:48:46 2011
@@ -548,7 +548,7 @@ DIR=`pwd`
 
   def _get_all_lib_deps(self, target_name, libdep_cache, paths):
     if not target_name in libdep_cache:
-      libs = {}
+      libs = set()
       path = None
       if target_name in self.sections:
         section = self.sections[target_name]
@@ -557,11 +557,10 @@ DIR=`pwd`
         if opt_libs:
           for lib_name in opt_libs.split():
             if lib_name.startswith('libsvn_'):
-              libs[lib_name] = True
+              libs.add(lib_name)
             for lib in self._get_all_lib_deps(lib_name, libdep_cache, paths):
-              libs[lib] = True
-      libdep_cache[target_name] = libs.keys()[:]
-      libdep_cache[target_name].sort()
+              libs.add(lib)
+      libdep_cache[target_name] = sorted(libs)
     return libdep_cache[target_name]
 
 class UnknownDependency(Exception):