Author: julianfoad
Date: Thu Jul 28 16:37:55 2011
New Revision: 1151916

URL: http://svn.apache.org/viewvc?rev=1151916&view=rev
Log:
Revert r1151672 and r1151680 which made the test suite fail to run all or
many of the Python tests while still reporting success.

The original combined log message of the two revs was effectively:

  Bump the Python tests' equivalent of SVN_VER_MINOR.  Since I'm too
  lazy to make this parse svn_version.h, opt for the least-maintenance
  solution:

  * subversion/tests/cmdline/svntest/main.py
    (_create_parser): Make 'server_minor_version' default to inf (positive
      infinity).
    (_parse_options): Account for the change.

Modified:
    subversion/trunk/subversion/tests/cmdline/svntest/main.py

Modified: subversion/trunk/subversion/tests/cmdline/svntest/main.py
URL: 
http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/svntest/main.py?rev=1151916&r1=1151915&r2=1151916&view=diff
==============================================================================
--- subversion/trunk/subversion/tests/cmdline/svntest/main.py (original)
+++ subversion/trunk/subversion/tests/cmdline/svntest/main.py Thu Jul 28 
16:37:55 2011
@@ -1516,7 +1516,7 @@ def _create_parser():
 
   # most of the defaults are None, but some are other values, set them here
   parser.set_defaults(
-        server_minor_version=float('+inf'), # positive infinity
+        server_minor_version=7,
         url=file_scheme_prefix + pathname2url(os.path.abspath(os.getcwd())),
         http_library=_default_http_library)
 
@@ -1540,8 +1540,7 @@ def _parse_options(arglist=sys.argv[1:])
 
   # If you change the below condition then change
   # ../../../../build/run_tests.py too.
-  if (options.server_minor_version < 3 or options.server_minor_version > 7) \
-      and options.server_minor_version != float('inf'):
+  if options.server_minor_version < 3 or options.server_minor_version > 7:
     parser.error("test harness only supports server minor versions 3-7")
 
   if options.url:


Reply via email to