GNU bug report logs - #49474
28.0.50; package-menu-filter-by-status doesn't accept argument it claims to

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 49474 in the body.
You can then email your comments to 49474 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: Adam Porter <adam <at> alphapapa.net>
To: bug-gnu-emacs <at> gnu.org
Subject: 28.0.50; package-menu-filter-by-status doesn't accept argument it
 claims to
Date: Thu, 8 Jul 2021 20:20:51 -0500

The docstring for the function package-menu-filter-by-status says:

    When called interactively, prompt for STATUS, which can be a
    comma-separated string.  If STATUS is empty, show all packages.

    When called from Lisp, STATUS can be a string or a list of
    strings.  If STATUS is nil or the empty string, show all
    packages.

However, when given a comma-separated string or a list of strings, it
doesn't work, because it applies a filter predicate that calls
string-match-p, which expects a regexp string.

Thanks for your work on Emacs.

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Adam Porter <adam <at> alphapapa.net>
Cc: 49474 <at> debbugs.gnu.org
Subject: Re: bug#49474: 28.0.50; package-menu-filter-by-status doesn't
 accept argument it claims to
Date: Sat, 10 Jul 2021 18:54:14 +0200

Adam Porter <adam <at> alphapapa.net> writes:

> The docstring for the function package-menu-filter-by-status says:
>
>     When called interactively, prompt for STATUS, which can be a
>     comma-separated string.  If STATUS is empty, show all packages.
>
>     When called from Lisp, STATUS can be a string or a list of
>     strings.  If STATUS is nil or the empty string, show all
>     packages.
>
> However, when given a comma-separated string or a list of strings, it
> doesn't work, because it applies a filter predicate that calls
> string-match-p, which expects a regexp string.

Yup.  I've now changed the command (in Emacs 28) to work as documented.
Hopefully nobody were relying on the previous, undocumented regexp-match
behaviour.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

Previous Next

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