GNU bug report logs - #6709
24.0.50; With semantic-mode enabled mo-git-blame-current gives a post-command-hook error

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 6709 in the body.
You can then email your comments to 6709 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: aneesh.kumar <at> linux.vnet.ibm.com
To: bug-gnu-emacs <at> gnu.org
Subject: 24.0.50; With semantic-mode enabled mo-git-blame-current gives a
	post-command-hook error
Date: Fri, 23 Jul 2010 01:13:20 +0530

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

In GNU Emacs 24.0.50.1 (i686-pc-linux-gnu, GTK+ Version 2.20.1)
 of 2010-07-22 on skywalker
Windowing system distributor `The X.Org Foundation', version 11.0.10706000
configured using `configure  '--prefix=/home/kvaneesh/bin-local/emacs/''

Important settings:
  value of $LC_ALL: nil
  value of $LC_COLLATE: nil
  value of $LC_CTYPE: nil
  value of $LC_MESSAGES: nil
  value of $LC_MONETARY: nil
  value of $LC_NUMERIC: nil
  value of $LC_TIME: nil
  value of $LANG: en_IN
  value of $XMODIFIERS: nil
  locale-coding-system: iso-latin-1-unix
  default enable-multibyte-characters: t

Major mode: Fundamental

Minor modes in effect:
  global-semanticdb-minor-mode: t
  global-semantic-idle-scheduler-mode: t
  semantic-mode: t
  scroll-all-mode: t
  tooltip-mode: t
  mouse-wheel-mode: t
  tool-bar-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  blink-cursor-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  line-number-mode: t
  transient-mark-mode: t

Recent input:
M-x l o a d - f i <tab> <return> e m <tab> m <backspace> 
/ m o <tab> <return> C-x C-f <backspace> <backspace> 
/ h <tab> o p <tab> s o <tab> k e <tab> l i <tab> 2 
<tab> / k e <tab> a c <tab> <return> SPC M-x m o - 
g i <tab> c u <tab> <return> <left> <down> <down> <down> 
<down> <down> <down> <down> <down> <down> <down> <down> 
<down> <down> <down> <down> <down> <down> <down> <down> 
<down> <down> <down> <down> <down> <down> <down> C-x 
<up> <up> q M-x s m e n <backspace> <backspace> <backspace> 
e m a n <tab> <return> C-h v [ <backspace> p o s <tab> 
c o <tab> <return> q <backspace> C-x 1 M-x <up> <up> 
<return> <down> <down> <down> <down> <down> <down> 
<down> <down> <down> <down> <down> <down> <down> <down> 
<down> <down> <down> <down> <down> <down> <down> <down> 
<down> <down> <down> <down> <down> <down> <down> <down> 
<down> C-x b * M e <tab> <return> C-x 1 M-x m x - r 
e <tab> <backspace> <backspace> <backspace> <backspace> 
<backspace> r e p o <tab> r <tab> <return>

Recent messages:
Loading semantic/db-file...done
Parsing acct.c (LL)...done
Type C-x 1 to delete the help window.
Running 'git cat-file'... done
Running 'git blame'...
Error in post-command-hook: (wrong-type-argument arrayp nil)
Git finished.
Running 'git blame'... done
Auto-saving...done
Making completion list...

Load-path shadows:
None found.

Features:
(shadow sort gnus-util mail-extr message sendmail rfc822 mml mml-sec
mm-decode mm-bodies mm-encode mail-parse rfc2231 rfc2047 rfc2045
ietf-drums mm-util mail-prsvr mailabbrev mail-utils gmm-utils mailheader
emacsbug semantic/tag-write semantic/find semantic/edit pp help-mode
view help-fns semantic/bovine cus-start cus-load semantic/db-file
cedet-files semantic/bovine/c semantic/decorate/include semantic/db-find
semantic/db-ref semantic/decorate/mode semantic/decorate pulse
semantic/bovine/c-by semantic/lex-spp semantic/bovine/gcc semantic/dep
semantic/analyze semantic/sort semantic/scope semantic/analyze/fcn
semantic/db-mode semantic/db eieio-base semantic/idle semantic/format
ezimage semantic/tag-ls semantic/ctxt semantic/util-modes semantic/util
semantic semantic/tag semantic/lex semantic/fw eieio byte-opt bytecomp
byte-compile mode-local cedet time-date vc-git cc-mode cc-fonts cc-menus
cc-cmds cc-styles cc-align cc-engine cc-vars cc-defs regexp-opt
mo-git-blame edmacro kmacro scroll-all easymenu cl cl-19 tooltip
ediff-hook vc-hooks lisp-float-type mwheel x-win x-dnd tool-bar dnd
fontset image fringe lisp-mode register page menu-bar rfn-eshadow timer
select scroll-bar mldrag mouse jit-lock font-lock syntax facemenu
font-core frame cham georgian utf-8-lang misc-lang vietnamese tibetan
thai tai-viet lao korean japanese hebrew greek romanian slovak czech
european ethiopic indian cyrillic chinese case-table epa-hook
jka-cmpr-hook help simple abbrev loaddefs button minibuffer faces
cus-face files text-properties overlay md5 base64 format env code-pages
mule custom widget hashtable-print-readable backquote
make-network-process dbusbind dynamic-setting font-render-setting gtk
x-toolkit x multi-tty emacs)

Message #8 received at 6709 <at> debbugs.gnu.org (full text, mbox):

From: Dan Nicolaescu <dann <at> gnu.org>
To: aneesh.kumar <at> linux.vnet.ibm.com
Cc: 6709 <at> debbugs.gnu.org
Subject: Re: bug#6709: 24.0.50;
	With semantic-mode enabled mo-git-blame-current gives a
	post-command-hook error
Date: Sat, 24 Jul 2010 01:23:15 -0400

aneesh.kumar <at> 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 ? 

Message #11 received at 6709 <at> debbugs.gnu.org (full text, mbox):

From: "Aneesh Kumar K. V" <aneesh.kumar <at> linux.vnet.ibm.com>
To: Dan Nicolaescu <dann <at> gnu.org>
Cc: 6709 <at> debbugs.gnu.org
Subject: Re: bug#6709: 24.0.50;
	With semantic-mode enabled mo-git-blame-current gives a
	post-command-hook error
Date: Mon, 26 Jul 2010 19:37:32 +0530

On Sat, 24 Jul 2010 01:23:15 -0400, Dan Nicolaescu <dann <at> gnu.org> wrote:
> aneesh.kumar <at> 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

1) No incremental blame support. That is we have to wait for the entire
history to be available.
2) ability to blame previous version/next version in the blame window
3) Ability to show the commit information on selecting the commit.
4) No scroll-all-window support so that boththe annotation information
and source code will scroll together


With respect to the bug. This bug happen only when semantic-mode is
enabled.

-aneesh

Message #14 received at 6709 <at> debbugs.gnu.org (full text, mbox):

From: Dan Nicolaescu <dann <at> gnu.org>
To: "Aneesh Kumar K. V" <aneesh.kumar <at> linux.vnet.ibm.com>
Cc: 6709 <at> debbugs.gnu.org
Subject: Re: bug#6709: 24.0.50;
	With semantic-mode enabled mo-git-blame-current gives a
	post-command-hook error
Date: Mon, 26 Jul 2010 17:41:26 -0400

"Aneesh Kumar K. V" <aneesh.kumar <at> linux.vnet.ibm.com> writes:

> On Sat, 24 Jul 2010 01:23:15 -0400, Dan Nicolaescu <dann <at> gnu.org> wrote:
>> aneesh.kumar <at> 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? 
You might want to file a bug report for this, maybe it will get implemented.

> 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...

Message #17 received at 6709 <at> debbugs.gnu.org (full text, mbox):

From: "Aneesh Kumar K. V" <aneesh.kumar <at> linux.vnet.ibm.com>
To: Dan Nicolaescu <dann <at> gnu.org>
Cc: 6709 <at> debbugs.gnu.org
Subject: Re: 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

On Mon, 26 Jul 2010 17:41:26 -0400, Dan Nicolaescu <dann <at> gnu.org> wrote:
> "Aneesh Kumar K. V" <aneesh.kumar <at> linux.vnet.ibm.com> writes:
> 
> > On Sat, 24 Jul 2010 01:23:15 -0400, Dan Nicolaescu <dann <at> gnu.org> wrote:
> >> aneesh.kumar <at> 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

Message #20 received at 6709 <at> debbugs.gnu.org (full text, mbox):

From: Dan Nicolaescu <dann <at> gnu.org>
To: "Aneesh Kumar K. V" <aneesh.kumar <at> linux.vnet.ibm.com>
Cc: 6709 <at> debbugs.gnu.org
Subject: Re: bug#6709: 24.0.50;
	With semantic-mode enabled mo-git-blame-current gives a
	post-command-hook error
Date: Tue, 27 Jul 2010 01:32:57 -0400

"Aneesh Kumar K. V" <aneesh.kumar <at> linux.vnet.ibm.com> writes:

> On Mon, 26 Jul 2010 17:41:26 -0400, Dan Nicolaescu <dann <at> gnu.org> wrote:
>> "Aneesh Kumar K. V" <aneesh.kumar <at> linux.vnet.ibm.com> writes:
>> 
>> > On Sat, 24 Jul 2010 01:23:15 -0400, Dan Nicolaescu <dann <at> gnu.org> wrote:
>> >> aneesh.kumar <at> linux.vnet.ibm.com writes:
>
> 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)

It should only be slow for the first file in a git tree, the following
ones should not show any observable delay.

Message #23 received at 6709 <at> debbugs.gnu.org (full text, mbox):

From: David Engster <deng <at> randomsample.de>
To: aneesh.kumar <at> linux.vnet.ibm.com
Cc: 6709 <at> debbugs.gnu.org
Subject: Re: 24.0.50; With semantic-mode enabled mo-git-blame-current gives a
	post-command-hook error
Date: Mon, 12 Mar 2012 20:53:58 +0100

aneesh kumar 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

I know it's been a while, but I currently cannot reproduce this. If
you're still seeing this error, please do M-x toggle-debug-on-error and
send the resulting backtrace.

Cheers,
David

Message #28 received at 6709-done <at> debbugs.gnu.org (full text, mbox):

From: David Engster <deng <at> randomsample.de>
To: aneesh.kumar <at> linux.vnet.ibm.com
Cc: 6709-done <at> debbugs.gnu.org
Subject: Re: 24.0.50; With semantic-mode enabled mo-git-blame-current gives a
	post-command-hook error
Date: Tue, 16 Oct 2012 22:30:37 +0200

David Engster writes:
> aneesh kumar 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
>
> I know it's been a while, but I currently cannot reproduce this. If
> you're still seeing this error, please do M-x toggle-debug-on-error and
> send the resulting backtrace.

Since I cannot reproduce this and no further information was given, I'm
closing this report.

-David

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.