On 2017/05/26 14:26, Paul Irofti wrote:
> On Fri, May 26, 2017 at 06:55:34AM -0400, Ted Unangst wrote:
> > let's say, hypothetically, you're working with a blasphemer who mails you a
> > git diff which looks a little like this:
> > 
> > diff --git a/d2/bar.c b/d2/bar.c
> > index 496ef02..51a910f 100644
> > --- a/d2/bar.c
> > +++ b/d2/bar.c
> 
> This is a stupid git setting indeed and I think it needs to be fixed by
> yelling at the user to issue
> 
> $ git config --global diff.noprefix true
> 
> The global flag is of course optional.

Making this a global setting is a problem for people who occasionally send
diffs to other projects, some of these get confused/annoyed if you start
sending them "weird" diffs without the a/ b/ prefixes ;)

> Our github mirror should include a .gitconfig file with that option set
> by default.

That doesn't get used automatically, users would still have to type
something to pull it in. (And in that case, it's probably better that
they just set it directly themselves, rather than trust a .gitconfig
from the repo..)

Reply via email to