Ignoring whitespace changes in 'svn blame'

This seems the right timing for a bit of designing.

My two-step, completely unresearched thought for 'svn blame'
is:

(a) Make 'svn blame' use the generic 'diff' facility (i.e.
internal or external 
diff, at the user's choice).  Give it the options
'--diff-cmd' and 
'-x/--extensions' (yuck, bad name).  This will immediately
give 'svn blame' the 
ability to ignore space changes and the like, by using an
external 'diff'.

Does anyone have an idea of what's involved in doing that?

(b) Give the internal 'diff' some options such as to ignore
whitespace changes. 
  This can be a completely independent enhancement.

- Julian

------------------------------------------------------------
---------
To unsubscribe, e-mail: dev-unsubscribesubversion.tigris.org
For additional commands, e-mail: dev-helpsubversion.tigris.org

Julian Foad wrote:
> This seems the right timing for a bit of designing.
>
> My two-step, completely unresearched thought for 'svn
blame' is:
>
> (a) Make 'svn blame' use the generic 'diff' facility
(i.e. internal or 
> external diff, at the user's choice).  Give it the
options 
> '--diff-cmd' and '-x/--extensions' (yuck, bad name). 
This will 
> immediately give 'svn blame' the ability to ignore
space changes and 
> the like, by using an external 'diff'.
>
> Does anyone have an idea of what's involved in doing
that?
Writing a parser for arbitrary diff formats, so that the
result of the 
diff can actually be converted into blame info? Really, this
idea seems 
seriously flawed.

> (b) Give the internal 'diff' some options such as to
ignore whitespace 
> changes.  This can be a completely independent
enhancement.
I think this is the only reasonable way to teach blame to
ignore 
whitespace changes.

-- Brane


------------------------------------------------------------
---------
To unsubscribe, e-mail: dev-unsubscribesubversion.tigris.org
For additional commands, e-mail: dev-helpsubversion.tigris.org