[SciPy-dev] SVN "corruption" / bzr-svn oddity?

[Pauli Virtanen]

Hi all,

Commit r5661 apparently reverted some previous commits: compare

	http://projects.scipy.org/scipy/browser/trunk/scipy/special/tests/test_basic.py?rev=5660
	http://projects.scipy.org/scipy/browser/trunk/scipy/special/tests/test_basic.py?rev=5661

looking at the line "cephes.pbdv(1,0),(0.0,0.0)", which previously read
"cephes.pbdv(1,0),(0.0,1.0)". The strange thing is that

	http://projects.scipy.org/scipy/changeset/5661

does not show that this change was made. It seems like bzr-svn did something
clever...


Moreover,

	svn diff -r 5660:5661 test_basic.py

says

	svn: Unable to find repository location for 'test_basic.py' in revision 5660

which is unexpected.


The Git repository indicates that something like this occurred:

       5655 5656 5657 5658 5659 5660
	o----o----o----o----o----o----o 5661
         \___________________________/

So, the revision 5661 is based on 5655. Now, bzr-svn has done something
and silently reverted the changesets 5656-5660 so that the reversion
does not appear in the commit 5661. (Surprisingly, git-svn *recognized* this
as a merge!) The full diff appears to be

	$ git diff --stat 746e23..svn/trunk
	INSTALL.txt                                       |  222 ++++++++++++++-------
	scipy/io/matlab/tests/data/test_skip_variable.mat |  Bin 20225 -> 0 bytes
	scipy/io/matlab/tests/test_mio.py                 |   31 +---
	scipy/special/specfun_wrappers.c                  |    6 +-
	scipy/special/tests/test_basic.py                 |   31 +---
	scipy/stats/distributions.py                      |    2 +-

So, it seems that bzr-svn does some "deep" SVN-fu on merges. Looking at
the commit message for r5661 more closely, it says ". (copied from trunk)",
and indeed "svn log ." indicates that r5661 is based on r5660.

Does someone understand SVN enough to know what happened and how
to revert it, if needed?

My guess would be

	svn cp http://svn.scipy.org/svn/scipy/trunk@5660 http://svn.scipy.org/svn/scipy/trunk

    ***

Wed, 15 Apr 2009 23:31:39 +0200, Xavier Gnata wrote:
[clip]
> Well I'm do not understand this point Looking at
> http://svn.scipy.org/svn/scipy/trunk/scipy/special/tests/test_basic.py I
> can read :
> 
> def test_pbdv(self):
>         assert_equal(cephes.pbdv(1,0),(0.0,0.0)
> 
> This website claims I'm looking at
> 
> 
>     Revision 5661: /trunk/scipy/special/tests
> 
> I'm must be missing something...sorry for the noise....

Good catch, you're completely right! Something strange is going on.
I only checked it via looking at the commits in

	http://projects.scipy.org/scipy/timeline

and as you can see, none of the commits after 5657 actually reverts the 
change, so I assumed it was still there...

-- 
Pauli Virtanen