Re: [PATCH] Add a test for issue#3870

2011-05-05 Thread Philip Martin
Arwin Arni writes: > * subversion/tests/cmdline/svnsync_tests.py > (setup_and_sync): Modified to accept two bools 'is_src_ra_local' and > 'is_dest_ra_local' to sync from or to local URLs. > > (run_test): Percolate is_src_ra_local and is_dest_ra_local. >

Re: [PATCH] Add a test for issue#3870

2011-05-05 Thread Philip Martin
Arwin Arni writes: > On Wednesday 04 May 2011 10:27 PM, Philip Martin wrote: >> Arwin Arni writes: >> >>> Index: subversion/tests/cmdline/svnsync_tests.py >>> === >>> --- subversion/tests/cmdline/svnsync_tests.py (revision 109

Re: [PATCH] Add a test for issue#3870

2011-05-04 Thread Philip Martin
Philip Martin writes: >> +@Issue(3870) >> +@SkipUnless(svntest.main.is_posix_os) >> +@XFail() And that should be: @XFail(svntest.main.is_ra_type_dav_serf) >> +def fd_leak_sync_from_serf_to_local(sbox): >> + "FD leak during sync over serf to local" -- Philip

Re: [PATCH] Add a test for issue#3870

2011-05-04 Thread Philip Martin
Arwin Arni writes: > Index: subversion/tests/cmdline/svnsync_tests.py > === > --- subversion/tests/cmdline/svnsync_tests.py (revision 1099373) > +++ subversion/tests/cmdline/svnsync_tests.py (working copy) > @@ -155,7 +155,8 @@ > >

[PATCH] Add a test for issue#3870

2011-05-04 Thread Arwin Arni
Hi All, This patch adds a test to svnsync_tests.py, to showcase issue #3870: (http://subversion.tigris.org/issues/show_bug.cgi?id=3870) "File descriptor leaks during svnsync" Regards, Arwin Arni Index: subversion/tests/cmdline/svnsync_tests.py ==