Add svndiff tool: runs skdiff over all pending changes in SVN checkout

Add svndiff tool: runs skdiff over all pending changes in SVN checkout

This is needed for 
https://goto.google.com/HowToLandSkiaChangesThatChangeWebkitLayoutTests ,
and will be generally useful beyond that too.
Committed: https://code.google.com/p/skia/source/detail?r=4052

[epoger, 2012-05-25 17:21:18]

first LG wins

[bungeman, 2012-05-25 18:00:18]

https://codereview.appspot.com/6256052/diff/1/tools/svndiff.py
File tools/svndiff.py (right):

https://codereview.appspot.com/6256052/diff/1/tools/svndiff.py#newcode98
tools/svndiff.py:98: original_file_path = os.path.join(
This magic will not work with svn past 1.7 as it uses a database at the top
level.

Perhaps something like pulling in the output of

svn cat -r BASE

would work? This isn't supposed to hit the server, see
http://subversion.tigris.org/issues/show_bug.cgi?id=1361 .

[epoger, 2012-05-25 19:26:28]

https://codereview.appspot.com/6256052/diff/1/tools/svndiff.py
File tools/svndiff.py (right):

https://codereview.appspot.com/6256052/diff/1/tools/svndiff.py#newcode98
tools/svndiff.py:98: original_file_path = os.path.join(
On 2012/05/25 18:00:18, bungeman wrote:
> This magic will not work with svn past 1.7 as it uses a database at the top
> level.
> 
> Perhaps something like pulling in the output of
> 
> svn cat -r BASE
> 
> would work? This isn't supposed to hit the server, see
> http://subversion.tigris.org/issues/show_bug.cgi?id=1361 .

Excellent point, thanks!  It turns out that "svn export -R BASE destpath" works.
 See change here and in svn.py.

[epoger, 2012-05-25 19:29:43]

On 2012/05/25 19:26:28, epoger wrote:
> https://codereview.appspot.com/6256052/diff/1/tools/svndiff.py
> File tools/svndiff.py (right):
> 
> https://codereview.appspot.com/6256052/diff/1/tools/svndiff.py#newcode98
> tools/svndiff.py:98: original_file_path = os.path.join(
> On 2012/05/25 18:00:18, bungeman wrote:
> > This magic will not work with svn past 1.7 as it uses a database at the top
> > level.
> > 
> > Perhaps something like pulling in the output of
> > 
> > svn cat -r BASE
> > 
> > would work? This isn't supposed to hit the server, see
> > http://subversion.tigris.org/issues/show_bug.cgi?id=1361 .
> 
> Excellent point, thanks!  It turns out that "svn export -R BASE destpath"
works.
>  See change here and in svn.py.

I mean, "svn export --revision BASE file_within_repo destpath"

[bungeman, 2012-05-25 19:41:39]

LGTM

On 2012/05/25 19:26:28, epoger wrote:
> https://codereview.appspot.com/6256052/diff/1/tools/svndiff.py
> File tools/svndiff.py (right):
> 
> https://codereview.appspot.com/6256052/diff/1/tools/svndiff.py#newcode98
> tools/svndiff.py:98: original_file_path = os.path.join(
> On 2012/05/25 18:00:18, bungeman wrote:
> > This magic will not work with svn past 1.7 as it uses a database at the top
> > level.
> > 
> > Perhaps something like pulling in the output of
> > 
> > svn cat -r BASE
> > 
> > would work? This isn't supposed to hit the server, see
> > http://subversion.tigris.org/issues/show_bug.cgi?id=1361 .
> 
> Excellent point, thanks!  It turns out that "svn export -R BASE destpath"
works.
>  See change here and in svn.py.