Hyrum K Wright wrote on Fri, Feb 11, 2011 at 18:09:14 +0000:
> On Fri, Feb 11, 2011 at 5:59 PM, Kamesh Jayachandran <kam...@collab.net>
> wrote:
> >>kame...@apache.org wrote on Tue, Feb 08, 2011 at 14:25:06 -0000:
> >>> + expected_error_for_cat = "svn: E175013: Unable to connect to a
> >>> repository"+ \
> >>> + " at URL '" + mu_url + "'\n" + \
> >>> + "svn: E175013: Access to '/" + mu_repo_path
> >>> + \
> >>> + "' forbidden\n" + \
> >>> + "svn: E175009: XML parsing failed: (403
> >>> Forbidden)"
> >
> >>Bad idea to hard code so much of the specific message we're generating
> >>today; it's bound to change later.
> >
> >>(eg, the XML parsing thing would ideally go away at some point)
> >
> >>Just 'Access.*${mu_url}.*forbidden' is enough IMO.
> >
> >
> > Thanks We fixed this in r1068802 Still we have "E170001:" for svnserve would
> > remove it as soon as I reach before my working copy.
Thanks. But why remove the E170001? Its very purpose is to provide
a machine-readable identification of the error... (exactly what tests
need)
