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):