Re: svn commit: rev 5374 - in trunk/subversion: clients/cmdline tests/clients/cmdline

cmpilato@tigris.org wrote:

> ########################################################################
> # Run the tests
>
>@@ -876,6 +927,13 @@
> # non-Posix platforms, we won't have to skip here:
> Skip(revprop_change, (os.name != 'posix')),
> prop_value_conversions,
>+ # This stuff currently fails over DAV, but I can't seem to
>+ # make a conditional for that (e.g., svntest.main.test_area_url
>+ # has not yet been populated at this point, so I can't check
>+ # whether or not it starts with 'http'). So, skip for all
>+ # RA layers until we turn on binary property support in
>+ # libsvn_ra_dav (see issue #1015).
>+ Skip(binary_props),
> ]
>
>

What's this, what's this? Committing without running the tests again,
are we? The Skip predicate needs a condition argument. If you want to
unconditionally skip the test, you do this:

    Skip(binary_props, 1)

-- 
Brane Čibej   <brane_at_xbc.nu>   http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org