GNU bug report logs - #14930
24.3.50; Reverting process list fails after list-packages

Previous Next

Package: emacs;

Reported by: Ari Roponen <ari.roponen <at> gmail.com>

Date: Mon, 22 Jul 2013 11:15:03 UTC

Severity: normal

Found in version 24.3.50

Done: Juanma Barranquero <lekktu <at> gmail.com>

Bug is archived. No further changes may be made.

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

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

View this report as an mbox folder, status mbox, maintainer mbox

Report forwarded to bug-gnu-emacs <at> gnu.org:
bug#14930; Package emacs. (Mon, 22 Jul 2013 11:15:03 GMT) Full text and rfc822 format available.
Acknowledgement sent to Ari Roponen <ari.roponen <at> gmail.com>:
New bug report received and forwarded. Copy sent to bug-gnu-emacs <at> gnu.org. (Mon, 22 Jul 2013 11:15:03 GMT) Full text and rfc822 format available.

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

From: Ari Roponen <ari.roponen <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 24.3.50; Reverting process list fails after list-packages
Date: Mon, 22 Jul 2013 14:13:56 +0300

Starting from emacs -Q:

  M-x list-packages
  M-x list-processes
  C-x o 				; Switch to the process list.
  g					; Revert buffer.

=> apply: Args out of range: [("ack" face link follow-link t package-desc ...

This happens because `package-mode-menu' adds a revert function to
`tabulated-list-revert-hook' globally, and reverting process list
tries to run it.

The following patch fixes the problem by adding the function to the hook
locally.


2013-07-22  Ari Roponen  <ari.roponen <at> gmail.com>

	* emacs-lisp/package.el (package-menu-mode): Don't modify the
	global value of tabulated-list-revert-hook.


=== modified file 'lisp/emacs-lisp/package.el'
--- lisp/emacs-lisp/package.el	2013-06-26 00:44:35 +0000
+++ lisp/emacs-lisp/package.el	2013-07-22 09:56:00 +0000
@@ -1393,7 +1393,7 @@
 			       ("Description" 0 nil)])
   (setq tabulated-list-padding 2)
   (setq tabulated-list-sort-key (cons "Status" nil))
-  (add-hook 'tabulated-list-revert-hook 'package-menu--refresh)
+  (add-hook 'tabulated-list-revert-hook 'package-menu--refresh nil t)
   (tabulated-list-init-header))
 
 (defmacro package--push (pkg-desc status listname)


In GNU Emacs 24.3.50.6 (x86_64-unknown-linux-gnu, GTK+ Version 3.9.8)
 of 2013-07-22 on arirop
Bzr revision: 113489 rgm <at> gnu.org-20130722102102-t430il8zjpfasx60

-- 
Ari Roponen
Reply sent to Juanma Barranquero <lekktu <at> gmail.com>:
You have taken responsibility. (Mon, 22 Jul 2013 11:42:02 GMT) Full text and rfc822 format available.
Notification sent to Ari Roponen <ari.roponen <at> gmail.com>:
bug acknowledged by developer. (Mon, 22 Jul 2013 11:42:03 GMT) Full text and rfc822 format available.

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

From: Juanma Barranquero <lekktu <at> gmail.com>
To: Ari Roponen <ari.roponen <at> gmail.com>
Cc: 14930-done <at> debbugs.gnu.org
Subject: Re: bug#14930: 24.3.50;
 Reverting process list fails after list-packages
Date: Mon, 22 Jul 2013 13:40:45 +0200

On Mon, Jul 22, 2013 at 1:13 PM, Ari Roponen <ari.roponen <at> gmail.com> wrote:

> 2013-07-22  Ari Roponen  <ari.roponen <at> gmail.com>
>
>         * emacs-lisp/package.el (package-menu-mode): Don't modify the
>         global value of tabulated-list-revert-hook.

Thanks, committed as revno:113490.
bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Tue, 20 Aug 2013 11:24:03 GMT) Full text and rfc822 format available.
This bug report was last modified 10 years and 263 days ago.

Previous Next

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