Re: problems with svn diff [file]

Karl Fogel <kfogel@newton.ch.collab.net> writes:

> Philip Martin <philip@codematters.co.uk> writes:
> > The only diff regression tests are the ones I wrote for
> > 'svn diff -r'. I forgot that these won't test plain 'svn diff' :-(
>
> Well, certainly not your fault -- I should have written a test for
> plain "svn diff" when I implemented it. If I implemented it, which I
> frankly can't even remember anymore. :-(
>
> > I know what the problem is.
>
> Whew.
>
> Should I be writing a regression test, or will you be writing it as
> part of the fix? (I.e., while you CYA, will you also be CMA? :-) )

Looking at the existing diff tests they are not very extensive. Just
some 'svn diff -rREV' and 'svn diff -rREV1:REV2' on the whole
repository. I'd never seen Python before so I had a bit of a struggle
to write something to test the diff output, it was so much work I gave
up after writing a few tests...

I'll check in the fix so that people can run 'svn diff file' then I'll
write some more tests, but feel free to contribute or even just look
at my clunky Python code :-)

In the meantime: if you know which revision you have checked out ('svn
st FILE') then 'svn diff -rREV FILE' will work.

-- 
Philip
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org