[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

[PATCH] Fixed CMD display for main.py

From: Edmund Wong <ed_at_kdtc.net>
Date: Fri, 07 Aug 2009 15:43:59 +0800


Browsing through the tests.log and came across a strange
occurrence of the following:

CMD: svnadmin.exedump svn-test-work/local_tmp/repos | \
       svnadmin.exeload svn-test-work/repositories/autoprop_tests-1 ...

(Similarly under Linux):

CMD: svnadmindump svn-test-work/local_tmp/repos | \
       svnadminload svn-test-work/repositories/autoprop_tests-1 ...

While it's aesthetic in nature, it would be easier to figure
out the commands that was being executed.


     Fix typo in Command Message that gets written to the
     tests.log. A space was missing between the command
     and its arguments.

     * subversion/tests/cmdline/svntest/main.py:
       (copy_repos): s/%s%s/%s %s/

     Patch by: Edmund Wong <ed {at} kdtc.net>



Index: subversion/tests/cmdline/svntest/main.py
--- subversion/tests/cmdline/svntest/main.py (revision 38611)
+++ subversion/tests/cmdline/svntest/main.py (working copy)
@@ -793,7 +793,7 @@
   if ignore_uuid:
     load_args = load_args + ['--ignore-uuid']
   if verbose_mode:
- sys.stdout.write('CMD: %s%s | %s%s\n' % (os.path.basename(svnadmin_binary),
+ sys.stdout.write('CMD: %s %s | %s %s\n' % (os.path.basename(svnadmin_binary),
                                             ' '.join(dump_args),
                                             ' '.join(load_args)))
Received on 2009-08-07 09:47:42 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.