C. Michael Pilato wrote:
> Not a comment on the patch itself, but the "trans" in trans_tests.py is
> short for "translation", not "transport".
Uh, embarrassing, no idea how I got transport in my mind..
[[[
Fix two translation tests failures when running against a https remote repository.
* subversion/tests/cmdline/trans_tests.py
(keywords_from_birth),
(keyword_expanded_on_checkout): Extend match to succeed with https.
Patch by: jpeters7677_at_gmx.de
]]]
Index: subversion/tests/cmdline/trans_tests.py
===================================================================
--- subversion/tests/cmdline/trans_tests.py (revision 31014)
+++ subversion/tests/cmdline/trans_tests.py (working copy)
@@ -275,7 +275,7 @@
fp = open(url_unexp_path, 'r')
lines = fp.readlines()
if not ((len(lines) == 1)
- and (re.match("\$URL: (http|file|svn|svn\\+ssh)://", lines[0]))):
+ and (re.match("\$URL: (http|https|file|svn|svn\\+ssh)://", lines[0]))):
print "URL expansion failed for", url_unexp_path
raise svntest.Failure
fp.close()
@@ -284,7 +284,7 @@
fp = open(url_exp_path, 'r')
lines = fp.readlines()
if not ((len(lines) == 1)
- and (re.match("\$URL: (http|file|svn|svn\\+ssh)://", lines[0]))):
+ and (re.match("\$URL: (http|https|file|svn|svn\\+ssh)://", lines[0]))):
print "URL expansion failed for", url_exp_path
raise svntest.Failure
fp.close()
@@ -577,7 +577,7 @@
fp = open(other_url_path, 'r')
lines = fp.readlines()
if not ((len(lines) == 1)
- and (re.match("\$URL: (http|file|svn|svn\\+ssh)://", lines[0]))):
+ and (re.match("\$URL: (http|https|file|svn|svn\\+ssh)://", lines[0]))):
print "URL expansion failed for", other_url_path
raise svntest.Failure
fp.close()
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-05-05 17:39:53 CEST