difflame improvements

2017-03-05 Thread Edmundo Carmona Antoranz
Hi! Since my last post the biggest improvement is the ability to detect that the user has requested a "reverse" analysis. Under "normal" circumstances a user would ask difflame to get the diff from an ancestor (call "difflame treeish1 treeish2" so that merge-ba

Re: difflame improvements

2017-02-18 Thread Edmundo Carmona Antoranz
On Fri, Feb 17, 2017 at 1:01 AM, Edmundo Carmona Antoranz wrote: > On Thu, Feb 16, 2017 at 11:17 PM, Jeff King wrote: > >> This isn't difflame's fault; that's what "git blame" tells you about >> that line. But since I already told difflame "v

Re: difflame improvements

2017-02-16 Thread Edmundo Carmona Antoranz
On Thu, Feb 16, 2017 at 11:17 PM, Jeff King wrote: > This isn't difflame's fault; that's what "git blame" tells you about > that line. But since I already told difflame "v2.6.5..HEAD", it would > probably make sense to similarly limit the blame to t

Re: difflame improvements

2017-02-16 Thread Jeff King
else { + gh_logf("prune", "%s Duplicate loose object pruned\n", + sha1_to_hex(sha1)); unlink_or_warn(path); + } return 0; } Running difflame on it says this: $ python /path/to/difflame.py v2.6.5..HEAD -- b

difflame improvements

2017-02-14 Thread Edmundo Carmona Antoranz
ariable that gives the default value for its --inter-hunk-context option. * vn/diff-ihc-config: diff: add interhunk context config option And this is not telling me the _real_ revision where the lines were _deleted_ so it's not very helpful, as Peff has already mentioned. Ru

Re: difflame

2017-02-02 Thread Edmundo Carmona Antoranz
On Thu, Feb 2, 2017 at 10:46 PM, Edmundo Carmona Antoranz wrote: > I have been "scripting around git blame --reverse" for some days now. > Mind taking a look? I've been working on blame-deletions for this. blame-deletions branch, that is Sorry for the previous top-posting.

Re: difflame

2017-02-02 Thread Edmundo Carmona Antoranz
I have been "scripting around git blame --reverse" for some days now. Mind taking a look? I've been working on blame-deletions for this. 22:41 $ ../difflame/difflame.py HEAD~5 HEAD diff --git a/file b/file index b414108..051c298 100644 --- a/file +++ b/file @@ -1,6 +1,9 @@ ^151335

Re: difflame

2017-01-30 Thread Edmundo Carmona Antoranz
he added lines (blame reverse to try to get as close as > possible with a single command in case I want to see what happened > with something that was deleted). If I could get blame information of > added/deleted lines in a single run, that would help a lot. > > Lo and behold: difflame.

Re: difflame

2017-01-30 Thread Edmundo Carmona Antoranz
anged and blame the added lines (blame reverse to try to get as close as possible with a single command in case I want to see what happened with something that was deleted). If I could get blame information of added/deleted lines in a single run, that would help a lot. Lo and behold: difflame.

Re: difflame

2017-01-30 Thread Jeff King
ame information as well (to see when something was added/removed). > > > > This is something I've wanted, too. The trickiest part, though, is > > blaming deletions, because git-blame only tracks the origin of content, > > not the origin of a change. > > Hmph, this is a

Re: difflame

2017-01-30 Thread Junio C Hamano
ve wanted, too. The trickiest part, though, is > blaming deletions, because git-blame only tracks the origin of content, > not the origin of a change. Hmph, this is a comment without looking at what difflame does internally, so you can ignore me if I am misunderstood what problem you are

Re: difflame

2017-01-27 Thread Jeff King
is blaming deletions, because git-blame only tracks the origin of content, not the origin of a change. For example, try this case: git init for i in $(seq 1 10); do echo $i >>file git add file git commit -m "add $i" done sed 4d tmp && mv tmp file git com

difflame

2017-01-17 Thread Edmundo Carmona Antoranz
nk it could be worth to be added into git main (perhaps into contrib?). If you want to give ir a try: https://github.com/eantoranz/difflame Just provide the two treeishs you would like to diff (no more parameters are accepted at the time) and you will get the diff along with blame. Running it righ