Author: hwright
Date: Mon Jun 13 15:04:07 2011
New Revision: 1135124

URL: http://svn.apache.org/viewvc?rev=1135124&view=rev
Log:
For release.py, only pass the verbose flag to a function, rather than the
entire set of args.

* tools/dist/release.py
  (run_script): Take the verbose flag, rather than then entire args.
  (build_env): Update callers.

Modified:
    subversion/trunk/tools/dist/release.py

Modified: subversion/trunk/tools/dist/release.py
URL: 
http://svn.apache.org/viewvc/subversion/trunk/tools/dist/release.py?rev=1135124&r1=1135123&r2=1135124&view=diff
==============================================================================
--- subversion/trunk/tools/dist/release.py (original)
+++ subversion/trunk/tools/dist/release.py Mon Jun 13 15:04:07 2011
@@ -56,8 +56,8 @@ def get_nullfile():
     # This is certainly not cross platform
     return open('/dev/null', 'w')
 
-def run_script(args, script):
-    if args.verbose:
+def run_script(verbose, script):
+    if verbose:
         stdout = None
         stderr = None
     else:
@@ -121,7 +121,7 @@ def build_env(base_dir, args):
     # build autoconf
     logging.info('Building autoconf')
     os.chdir(os.path.join(get_tempdir(base_dir), params['autoconf']))
-    run_script(args,
+    run_script(args.verbose,
                '''./configure --prefix=%s
                   make
                   make install''' % get_prefix(base_dir))
@@ -129,7 +129,7 @@ def build_env(base_dir, args):
     # build libtool
     logging.info('Building libtool')
     os.chdir(os.path.join(get_tempdir(base_dir), params['libtool']))
-    run_script(args,
+    run_script(args.verbose,
                '''./configure --prefix=%s
                   make
                   make install''' % get_prefix(base_dir))
@@ -137,7 +137,7 @@ def build_env(base_dir, args):
     # build swig
     logging.info('Building swig')
     os.chdir(os.path.join(get_tempdir(base_dir), params['swig']))
-    run_script(args,
+    run_script(args.verbose,
                '''./configure --prefix=%s --without-pcre
                   make
                   make install''' % get_prefix(base_dir))


Reply via email to