Re: [PATCH (GITK,GIT-GUI,DOCS) 2/7] gitk: Support calling git gui blame from gitk.

From: Johannes Sixt <j.sixt@viscovery.net>
Date: Aug 26, 2008, 12:10 UTC
Message-ID: <48B3F2B9.4030403@viscovery.net>
In-Reply-To: <18611.55963.306938.896762@cargo.ozlabs.ibm.com>

Paul Mackerras schrieb:

Show 12 quoted lines

> Alexander Gavrilov writes:
> 
>> Add a new item to the file list popup menu, that
>> calls git gui blame for the selected file, starting
>> with the first parent of the current commit.
> 
> Also applied, also with some edits to the description.
> 
>> +    $flist_menu add command -label [mc "Blame parent commit"] \
>> +        -command {external_blame 1}
> 
> Why the parent commit rather than the current commit?

For the lines that the current commit *adds*, you know who is to blame (the current commit). Once you know that, you are usually interested in who is to blame for the lines that the current commit *removed*. To get this information, you must "Blame the parent commit".

-- Hannes