You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Jens Peters <jp...@gmx.de> on 2009/04/08 20:44:30 UTC

[PATCH] Fix a translation test that would fail when run over https://.

Hello,

This small patch fixes a glitch when running the testsuite over https.

Regards,
Jens

[[[
Fix a translation test that would fail when run over https://.

* subversion/tests/cmdline/trans_tests.py
   (keywords_from_birth): Add "https" to the regexp of patterns to match.

Patch by: <jpeters7677 {at} gmx.de>
]]]

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1601928

Re: [PATCH] Fix a translation test that would fail when run over https://.

Posted by Daniel Shahaf <d....@daniel.shahaf.name>.
Committed as r37316, with minor modification to meet the line width limit.

Thanks,

Daniel

Gavin Baumanis wrote on Fri, 17 Apr 2009 at 12:20 +1000:
> Ping. This patch submission has received no comments.
> 
> Gavin.
> 
> 
> On 09/04/2009, at 6:44 AM, Jens Peters wrote:
> 
> > Hello,
> >
> > This small patch fixes a glitch when running the testsuite over https.
> >
> > Regards,
> > Jens
> >
> > [[[
> > Fix a translation test that would fail when run over https://.
> >
> > * subversion/tests/cmdline/trans_tests.py
> >   (keywords_from_birth): Add "https" to the regexp of patterns to  
> > match.
> >
> > Patch by: <jpeters7677 {at} gmx.de>
> > ]]]
> >
> > ------------------------------------------------------
> > http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1601928Index 
> > : trans_tests.py
> > ===================================================================
> > --- trans_tests.py	(revision 37116)
> > +++ trans_tests.py	(working copy)
> > @@ -328,7 +328,7 @@
> >   fp = open(header_unexp_path, 'r')
> >   lines = fp.readlines()
> >   if not ((len(lines) == 1)
> > -          and (re.match("\$Header: (http|file|svn|svn\\+ssh)://.*  
> > jrandom",
> > +          and (re.match("\$Header: (http|https|file|svn|svn\\ 
> > +ssh)://.* jrandom",
> >                         lines[0]))):
> >     print("Header expansion failed for %s" % header_unexp_path)
> >     raise svntest.Failure
> > @@ -338,7 +338,7 @@
> >   fp = open(header_exp_path, 'r')
> >   lines = fp.readlines()
> >   if not ((len(lines) == 1)
> > -          and (re.match("\$Header: (http|file|svn|svn\\+ssh)://.*  
> > jrandom",
> > +          and (re.match("\$Header: (http|https|file|svn|svn\\ 
> > +ssh)://.* jrandom",
> >                         lines[0]))):
> >     print("Header expansion failed for %s" % header_exp_path)
> >     raise svntest.Failure
> 
> ------------------------------------------------------
> http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1759175
>

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1759416

Re: [PATCH] Fix a translation test that would fail when run over https://.

Posted by Gavin Baumanis <ga...@thespidernet.com>.
Ping. This patch submission has received no comments.

Gavin.


On 09/04/2009, at 6:44 AM, Jens Peters wrote:

> Hello,
>
> This small patch fixes a glitch when running the testsuite over https.
>
> Regards,
> Jens
>
> [[[
> Fix a translation test that would fail when run over https://.
>
> * subversion/tests/cmdline/trans_tests.py
>   (keywords_from_birth): Add "https" to the regexp of patterns to  
> match.
>
> Patch by: <jpeters7677 {at} gmx.de>
> ]]]
>
> ------------------------------------------------------
> http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1601928Index 
> : trans_tests.py
> ===================================================================
> --- trans_tests.py	(revision 37116)
> +++ trans_tests.py	(working copy)
> @@ -328,7 +328,7 @@
>   fp = open(header_unexp_path, 'r')
>   lines = fp.readlines()
>   if not ((len(lines) == 1)
> -          and (re.match("\$Header: (http|file|svn|svn\\+ssh)://.*  
> jrandom",
> +          and (re.match("\$Header: (http|https|file|svn|svn\\ 
> +ssh)://.* jrandom",
>                         lines[0]))):
>     print("Header expansion failed for %s" % header_unexp_path)
>     raise svntest.Failure
> @@ -338,7 +338,7 @@
>   fp = open(header_exp_path, 'r')
>   lines = fp.readlines()
>   if not ((len(lines) == 1)
> -          and (re.match("\$Header: (http|file|svn|svn\\+ssh)://.*  
> jrandom",
> +          and (re.match("\$Header: (http|https|file|svn|svn\\ 
> +ssh)://.* jrandom",
>                         lines[0]))):
>     print("Header expansion failed for %s" % header_exp_path)
>     raise svntest.Failure

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1759175