unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#6709: 24.0.50; With semantic-mode enabled mo-git-blame-current gives a post-command-hook error
@ 2010-07-22 19:43 aneesh.kumar
  2010-07-24  5:23 ` Dan Nicolaescu
  2012-03-12 19:53 ` David Engster
  0 siblings, 2 replies; 8+ messages in thread
From: aneesh.kumar @ 2010-07-22 19:43 UTC (permalink / raw)
  To: 6709


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)





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6709: 24.0.50; With semantic-mode enabled mo-git-blame-current gives a post-command-hook error
  2010-07-22 19:43 bug#6709: 24.0.50; With semantic-mode enabled mo-git-blame-current gives a post-command-hook error aneesh.kumar
@ 2010-07-24  5:23 ` Dan Nicolaescu
  2010-07-26 14:07   ` Aneesh Kumar K. V
  2012-03-12 19:53 ` David Engster
  1 sibling, 1 reply; 8+ messages in thread
From: Dan Nicolaescu @ 2010-07-24  5:23 UTC (permalink / raw)
  To: aneesh.kumar; +Cc: 6709

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 ? 





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6709: 24.0.50; With semantic-mode enabled mo-git-blame-current gives a post-command-hook error
  2010-07-24  5:23 ` Dan Nicolaescu
@ 2010-07-26 14:07   ` Aneesh Kumar K. V
  2010-07-26 21:41     ` Dan Nicolaescu
  0 siblings, 1 reply; 8+ messages in thread
From: Aneesh Kumar K. V @ 2010-07-26 14:07 UTC (permalink / raw)
  To: Dan Nicolaescu; +Cc: 6709

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

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





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6709: 24.0.50; With semantic-mode enabled mo-git-blame-current gives a post-command-hook error
  2010-07-26 14:07   ` Aneesh Kumar K. V
@ 2010-07-26 21:41     ` Dan Nicolaescu
  2010-07-27  5:15       ` Aneesh Kumar K. V
  0 siblings, 1 reply; 8+ messages in thread
From: Dan Nicolaescu @ 2010-07-26 21:41 UTC (permalink / raw)
  To: Aneesh Kumar K. V; +Cc: 6709

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





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6709: 24.0.50; With semantic-mode enabled mo-git-blame-current gives a post-command-hook error
  2010-07-26 21:41     ` Dan Nicolaescu
@ 2010-07-27  5:15       ` Aneesh Kumar K. V
  2010-07-27  5:32         ` Dan Nicolaescu
  0 siblings, 1 reply; 8+ messages in thread
From: Aneesh Kumar K. V @ 2010-07-27  5:15 UTC (permalink / raw)
  To: Dan Nicolaescu; +Cc: 6709

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





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6709: 24.0.50; With semantic-mode enabled mo-git-blame-current gives a post-command-hook error
  2010-07-27  5:15       ` Aneesh Kumar K. V
@ 2010-07-27  5:32         ` Dan Nicolaescu
  0 siblings, 0 replies; 8+ messages in thread
From: Dan Nicolaescu @ 2010-07-27  5:32 UTC (permalink / raw)
  To: Aneesh Kumar K. V; +Cc: 6709

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

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





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6709: 24.0.50; With semantic-mode enabled mo-git-blame-current gives a post-command-hook error
  2010-07-22 19:43 bug#6709: 24.0.50; With semantic-mode enabled mo-git-blame-current gives a post-command-hook error aneesh.kumar
  2010-07-24  5:23 ` Dan Nicolaescu
@ 2012-03-12 19:53 ` David Engster
  2012-10-16 20:30   ` David Engster
  1 sibling, 1 reply; 8+ messages in thread
From: David Engster @ 2012-03-12 19:53 UTC (permalink / raw)
  To: aneesh.kumar; +Cc: 6709

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





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6709: 24.0.50; With semantic-mode enabled mo-git-blame-current gives a post-command-hook error
  2012-03-12 19:53 ` David Engster
@ 2012-10-16 20:30   ` David Engster
  0 siblings, 0 replies; 8+ messages in thread
From: David Engster @ 2012-10-16 20:30 UTC (permalink / raw)
  To: aneesh.kumar; +Cc: 6709-done

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





^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2012-10-16 20:30 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-07-22 19:43 bug#6709: 24.0.50; With semantic-mode enabled mo-git-blame-current gives a post-command-hook error aneesh.kumar
2010-07-24  5:23 ` Dan Nicolaescu
2010-07-26 14:07   ` Aneesh Kumar K. V
2010-07-26 21:41     ` Dan Nicolaescu
2010-07-27  5:15       ` Aneesh Kumar K. V
2010-07-27  5:32         ` Dan Nicolaescu
2012-03-12 19:53 ` David Engster
2012-10-16 20:30   ` David Engster

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).