Re: [PATCH] Extra options for libsvn_diff

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.

> As an aside, I think the same about having a built-in default for
> "global-ignores" that matches some particular file extensions that we
> developers like to ignore ("*.o *.lo *.la #*# .*.rej *.rej .*~ *~ .#*
> .DS_Store"). In version 2 we should remove the built-in default and let
> it be configured entirely by the external configuration facility
> (~/.subversion/config).

In this case, Subversion inherited the default ignores from CVS (with
the addition of .DS_Store later). So at least there's another version
control tool to place blame on. :-)

-- 
C. Michael Pilato <cmpilato@collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand