bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#6709: 24.0.50; With semantic-mode enabled mo-git-blame-current gives


From: Aneesh Kumar K. V
Subject: bug#6709: 24.0.50; With semantic-mode enabled mo-git-blame-current gives a post-command-hook error
Date: Tue, 27 Jul 2010 10:45:17 +0530
User-agent: Notmuch/0.3.1-58-g6607fd6 (http://notmuchmail.org) Emacs/24.0.50.1 (i686-pc-linux-gnu)

On Mon, 26 Jul 2010 17:41:26 -0400, Dan Nicolaescu <dann@gnu.org> wrote:
> "Aneesh Kumar K. V" <aneesh.kumar@linux.vnet.ibm.com> writes:
> 
> > On Sat, 24 Jul 2010 01:23:15 -0400, Dan Nicolaescu <dann@gnu.org> wrote:
> >> aneesh.kumar@linux.vnet.ibm.com writes:
> >> 
> >> > Here is how to reproduce
> >> >
> >> > mo-git-blame can be downloaded from 
> >> >
> >> > git://git.bunkus.org/mo-git-blame.git
> >> >
> >> > emacs -Q
> >> > M-x load-file mo-git-blame-el
> >> > open a file in git repository
> >> > M-x mo-git-blame-current
> >> >
> >> > We will see the error 
> >> >
> >> > Running 'git cat-file'... done
> >> > Running 'git blame'...
> >> > Error in post-command-hook: (wrong-type-argument arrayp nil)
> >> > Git finished.
> >> > Running 'git blame'... done
> >> >
> >> > The error actually disable scroll-all-mode. So further scrolling doesn't
> >> > cause both windows to scroll
> >> 
> >> Not really related to your bug report, but from the name it looks like 
> >> mo-git-blame is a package for
> >> using "git blame".  Can you please tell us what it has that we are
> >> missing in C-x v g ? 
> >
> > I quickly tried C-x v g. This is what i found
> 
> Thanks for doing this!
> 
> 
> > 1) No incremental blame support. That is we have to wait for the entire
> > history to be available.
> 
> Does this make a difference in real life? 

Yes.

> You might want to file a bug report for this, maybe it will get
> implemented.

Will do

> 
> > 2) ability to blame previous version/next version in the blame window
> 
> Isn't that functionality available using the p n a j w keys? Please
> see the VC-annotate menu for the command names.  These commands should
> work even in the presence of renamed and moved files.
> 
> > 3) Ability to show the commit information on selecting the commit.
> 
> You can get the diff (using D) and the log (using L).  This might need
> another bug report, if git users think it's important it can be
> implemented in about 3-4 lines of code...
> 
> > 4) No scroll-all-window support so that boththe annotation information
> > and source code will scroll together
> 
> The two-column.el package should help doing that...
> 
> Maybe we need to ask the author of mo-git-blame-el if he wants to
> implement these features in vc-annotate.el...


I didn't really read the help to find whether these features are
supported. Now I find that i should be able to use vc-annotate.el for
all the my usecase. But i find that with vc enabled an open on the file
takes considerably more time than with vc disabled using

(setq vc-handled-backends nil)

-aneesh





reply via email to

[Prev in Thread] Current Thread [Next in Thread]