[debbugs-tracker] bug#14930: closed (24.3.50; Reverting process list fails after list-packages)

[GNU bug Tracking System]

Your message dated Mon, 22 Jul 2013 13:40:45 +0200
with message-id <address@hidden>
and subject line Re: bug#14930: 24.3.50; Reverting process list fails after 
list-packages
has caused the debbugs.gnu.org bug report #14930,
regarding 24.3.50; Reverting process list fails after list-packages
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
14930: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=14930
GNU Bug Tracking System
Contact address@hidden with problems
> --- Begin Message ---
>
>
>
> 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  <address@hidden>
>
>         * 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 address@hidden
>
> -- 
> Ari Roponen
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> 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 <address@hidden> wrote:
>
> > 2013-07-22  Ari Roponen  <address@hidden>
> >
> >         * emacs-lisp/package.el (package-menu-mode): Don't modify the
> >         global value of tabulated-list-revert-hook.
>
> Thanks, committed as revno:113490.
>
>
> --- End Message ---