On Wed, 2008-08-20 at 16:05 -0400, Karl Fogel wrote:
> Julian Foad <julianfoad_at_btopenworld.com> writes:
> > Julian Foad wrote:
> >> Does anyone know a reason not to remove all of this kludge?
> >
> > Here is a patch you can test with, especially if you run your tests with
> > a RAM disk or other file system, or if your CWD is read-only when
> > starting an individual test.
> >
> > If merge_tests.py 1 through 50 still pass with this demo (which disables
> > the kludge in those tests), then the principle is validated.
> >
> > Let us know whether it's OK!
>
> I think I've had a TODO item for, like, two years to check and see if
> that kludge is still necessary (I also suspect it's not). If your patch
> passes, +1 on getting rid of that thing. The code will get much
> cleaner, although, of course, that will be some effort, since many of
> the tests chdir and whatnot to cooperate with the kludge.
Done. Committed revision 32722.
- Julian
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-08-26 15:34:43 CEST