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

Re: svn commit: r1088602 - in /subversion/trunk/subversion: mod_dav_svn/mirror.c tests/cmdline/dav-mirror-autocheck.sh

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Tue, 5 Apr 2011 00:42:52 +0300

kameshj_at_apache.org wrote on Mon, Apr 04, 2011 at 12:55:38 -0000:
> @@ -438,5 +434,29 @@ fi
>
> say "PASS: master, slave are both at r4, as expected"
>
> -exit 0
> +# The following test case is for the regression issue triggered by r917523.
> +# The revision r917523 do some url encodings to the paths and uris which are
> +# not url-encoded. But there is one additional url-encoding of an uri which is
> +# already encoded. With this extra encoding, committing a path to slave which
> +# has space in it fails. Please see this thread
> +# http://svn.haxx.se/dev/archive-2011-03/0641.shtml for more info.
> +
> +say "Test case for regression issue triggered by r917523"
> +
> +$svnmucc cp 2 "$BASE_URL/trunk" "$BASE_URL/branch new"
> +$svnmucc put /dev/null "$BASE_URL/branch new/file" \
> +--config-option servers:global:http-library=neon
> +RETVAL=$?
> +

This looks bogus, why are you testing neon xor serf rather than both?

> +if [ $RETVAL -eq 0 ] ; then
> + say "PASS: committing a path which has space in it passes"
> +else
> + say "FAIL: committing a path which has space in it fails as there are extra
> + url-encodings happening in server side"
> +fi
Received on 2011-04-04 23:43:28 CEST

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.