On Thu, 16 Feb 2006, C. Michael Pilato wrote:
> Julian Foad wrote:
> > I know how useful the "--show-c-function" feature is, and I use it
> > myself when preparing patches for you most of the time, but I'm sorry, I
> > really can't accept it being built in to Subversion. I hate it being
> > built in to GNU "diff" (where it is) and I have been among the
> > developers refusing requests to add it to GNU "grep".
>
> +1.
>
Since ignore-whitespace/eol-style are the important things I want to
accomplish; i remove -p from the patch for now. People can use an
external diff tool for -p, but for blame and merge, it is not that easy:
- blame does not use an external diff (and we don't want to go down that
slippery slope parsing external diff output)
- The standard diff3 has no options to ignore whitespace changes.
Best,
//Peter
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Feb 16 21:24:25 2006