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

Re: Input validation observations

From: Julian Foad <julian.foad_at_wandisco.com>
Date: Mon, 20 Dec 2010 10:56:05 +0000

On Mon, 2010-12-13, Noorul Islam K M wrote:
> Julian Foad <julian.foad_at_wandisco.com> writes:
> >
> > * "svn mv ^/ ^/" -> "Cannot move path
> > 'https:/svn.apache.org/repos/asf' into itself"; should not print the URL
> > as if it's a local path.
>
> Attached is the patch which improves error message displayed.

> [[[
> Make 'svn move' display correct error message while moving any
> path/URL onto or into itself. Add missing tests.
>
> * subversion/libsvn_client/copy.c
> (try_copy): Display error message based on source type,
> (repos_to_repos_copy): Remove redundant code.
>
> * subversion/tests/cmdline/copy_tests.py
> (move_wc_and_repo_dir_to_itself, test_list): New test
>
> Patch by: Noorul Islam K M <noorul{_AT_}collab.net>
> ]]]

Committed revision 1051059. Thanks.

- Julian
Received on 2010-12-20 11:56:49 CET

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.