threads / discuss / 42787

git patch format for rename

Subject: git patch format for rename

## tl;dr

2 messages between Jul 6, 2016 and Jul 6, 2016.

replies: 1people: 2as markdown or json

Nadya.Zabrodina@jetbrains.com· Jul 6, 2016, 16:00 UTC · lore
Hi,
could you please explain to me why  ‘rename from & rename to’ information is added to git patch? 
Here is the patch created by `git format-patch`: 
    diff --git a/before.txt b/after.txt
    similarity index 100%
    rename from before.txt
    rename to after.txt
Is there any reason why git can’t recognize and apply rename-change which looks like this:
    diff --git a/before.txt b/after.txt
    --- a/before.txt
    +++ b/after.txt
    
or even like this:
    diff --git a/before.txt b/after.txt
I’m implementing a program (with an IDE) to create patches compatible with Git and want to have better understanding of the format and possible issues with it.
Thanks a lot in advance.
Stefan Beller· Jul 6, 2016, 17:45 UTC · re: Nadya.Zabrodina@jetbrains.com · lore

Re: git patch format for rename

On Wed, Jul 6, 2016 at 9:00 AM, Nadya.Zabrodina@jetbrains.com <Nadya.Zabrodina@jetbrains.com> wrote:

Show 8 quoted lines
> Hi,
>
> could you please explain to me why  ‘rename from & rename to’ information is added to git patch?
> Is there any reason why git can’t recognize and apply rename-change which looks like this:
>
>     diff --git a/before.txt b/after.txt
>     --- a/before.txt
>     +++ b/after.txt

See https://github.com/git/git/blob/master/Documentation/diff-generate-patch.txt for the status quo. I think most of it grew over time and made sense at the time of implementation, so the alternatives were maybe considered, you'd need to look for the old discussions around the diff format.

Show 10 quoted lines
>
> I’m implementing a program (with an IDE) to create patches compatible with Git and want to have better understanding of the format and possible issues with it.
>
> Thanks a lot in advance.
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe git" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

← back to recent threads