You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Greg Stein <gs...@gmail.com> on 2009/02/27 11:27:19 UTC

Re: svn commit: r36169 - trunk/subversion/tests/cmdline

Note that if you have *both* RA layers built into svn, then serf
should be chosen as the default (for the past few weeks). You may have
a stray override in your preferences. Point is: if anything, the
README should be more about "if you want Neon..." :-)

On Thu, Feb 26, 2009 at 23:46, Ben Collins-Sussman <su...@red-bean.com> wrote:
> Author: sussman
> Date: Thu Feb 26 14:46:44 2009
> New Revision: 36169
>
> Log:
> * subversion/tests/cmdline/README:  mention how to test with serf instead of neon.
>
> Modified:
>   trunk/subversion/tests/cmdline/README
>
> Modified: trunk/subversion/tests/cmdline/README
> URL: http://svn.collab.net/viewvc/svn/trunk/subversion/tests/cmdline/README?pathrev=36169&r1=36168&r2=36169
> ==============================================================================
> --- trunk/subversion/tests/cmdline/README       Thu Feb 26 14:44:22 2009        (r36168)
> +++ trunk/subversion/tests/cmdline/README       Thu Feb 26 14:46:44 2009        (r36169)
> @@ -134,6 +134,17 @@ Note [1]: It would be quite too much to
>           ----------------------------
>
>
> +[If you want to test with serf instead of neon:
> +
> +  $ ./basic_tests.py --url=http://localhost --http-library=serf
> +
> +  or
> +
> +  $ make check BASE_URL=http://localhost HTTP_LIBRARY=serf
> +]
> +
> +
> +
>  Running over ra_svn
>  -------------------
>
> ------------------------------------------------------
> http://subversion.tigris.org/ds/viewMessage.do?dsForumId=495&dsMessageId=1235266
>

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


Re: svn commit: r36169 - trunk/subversion/tests/cmdline

Posted by Greg Stein <gs...@gmail.com>.
On Fri, Feb 27, 2009 at 14:32, C. Michael Pilato <cm...@collab.net> wrote:
> Greg Stein wrote:
>> Note that if you have *both* RA layers built into svn, then serf
>> should be chosen as the default (for the past few weeks). You may have
>> a stray override in your preferences. Point is: if anything, the
>> README should be more about "if you want Neon..." :-)
>
> Stray overrides don't affect the test suite, by design.  Our test framework
> writes out its own temporary, custom runtime configuration area to use.  A
> quick check of that logic seems to show that it writes the 'servers' file
> with an http-library setting only if --http-library was passed to the tests,
> so this switch to Serf as the default WebDAV RA *should* have "taken" in the
> tests, too.

Oh. Good point!

I wonder if Ben was testing with serf and just didn't realize it? :-P

Cheers,
-g

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


Re: svn commit: r36169 - trunk/subversion/tests/cmdline

Posted by "C. Michael Pilato" <cm...@collab.net>.
Greg Stein wrote:
> Note that if you have *both* RA layers built into svn, then serf
> should be chosen as the default (for the past few weeks). You may have
> a stray override in your preferences. Point is: if anything, the
> README should be more about "if you want Neon..." :-)

Stray overrides don't affect the test suite, by design.  Our test framework
writes out its own temporary, custom runtime configuration area to use.  A
quick check of that logic seems to show that it writes the 'servers' file
with an http-library setting only if --http-library was passed to the tests,
so this switch to Serf as the default WebDAV RA *should* have "taken" in the
tests, too.

-- 
C. Michael Pilato <cm...@collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

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