From: Junio C Hamano <gitster@pobox.com>
Date: Wed, 16 Jan 2008 00:19:43 GMT
Subject: Re: [FEATURE REQUEST] git-svn format-patch
Message-ID: <7vhched3kw.fsf@gitster.siamese.dyndns.org>
In-Reply-To: <alpine.LNX.1.00.0801151728120.13593@iabervon.org>

Daniel Barkalow <barkalow@iabervon.org> writes:

> When --no-prefix is used, we should probably do:
>
> Index: <filename>
>
> instead of
>
> diff --git <filename> <filename>
>
> If nothing else, --no-prefix generates patches that git-apply can't apply 
> but thinks that it should be able to because of the "diff --git" line.

While I do not necessarily agree with that "Index: <filename>"
thing, I think dropping "--git" from there is probably a good
idea, as that is clearly not "--git" patch meant to be fed to
git-apply.

Actually I vaguely recall somebody suggested that we drop
"--git" if any nonstandard --src-prefix and --dst-prefix, but
sorry I do not recall the details (I am a bit sick today).  I
guess somehow we did not heed that wise advise and accepted the
change as-is, and that new feature ended up with a half-baked
hack that did not consider its consequences X-<.