unofficial mirror of help-gnu-emacs@gnu.org
 help / color / mirror / Atom feed
From: John Cummings <john@rootabega.net>
To: help-gnu-emacs@gnu.org, gnu-emacs-sources@gnu.org
Subject: Re: [GNU ELPA] Map version 3.2
Date: Thu, 23 Sep 2021 19:17:28 +0000	[thread overview]
Message-ID: <0Ww226bb9COlx-Xd2_M8rgS9220VEQgQRuds0to9jte2TIRzsawQh9bHwsGHwYV_tDoA-1Xem_17CMQwZuMaGdn7x2x9nkbyKZyKtyvcOZs=@rootabega.net> (raw)
In-Reply-To: <85ilyt3alu.fsf@elpa.gnu.org>


On Tuesday, September 21st, 2021 at 5:03 PM, ELPA update <do.not.reply@elpa.gnu.org> wrote:

> Version 3.2 of package Map has just been released in GNU ELPA.

Thanks for the update! I haven't knowingly used this package before,
so I downloaded it. It looks like map 3.1 and 3.2 might not be fully
compatible with Emacs 26 or 27. Has anyone else experienced this?

I first noticed it here, on my 26.3 Ubuntu build of Emacs, when I
visited a place with dir locals, and got this error:

Error reading dir-locals: (cl-no-applicable-method map-into
#s(hash-table size 65 test equal rehash-size 1.5 rehash-threshold
0.8125 data (nil ((tab-width . 8) (sentence-end-double-space . t)
(fill-column . 70) (bug-reference-url-format
. "https://debbugs.gnu.org/%s")))) list)

It looks like commit 6535fd1fa9ac21238a168916249ac59677a6118e allowed
generic specializers like "(eql 'list)", but since map.el updated in
the same commit, that method was no longer backwards compatible in the
subsequent releases (3.1 and 3.2)

I confirmed that calling the same map-into method from map.el got the
same cl-no-applicable-method error in a fresh repo build of Emacs 27.2
as well.

I might have gotten something really wrong here, but if not, does this
qualify as something to open a bug for? Do you think that the package
will just have its required Emacs version bumped and republished, if
that's possible? As I mentioned, I'm not too familiar with using
map.el, let alone developing it, but it seems like unquoted
specializers are still allowed in master, so would updating it to be
backwards-compatible be on the table?

Thank you!





       reply	other threads:[~2021-09-23 19:17 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <85ilyt3alu.fsf@elpa.gnu.org>
2021-09-23 19:17 ` John Cummings [this message]
2021-09-27  1:39   ` [GNU ELPA] Map version 3.2 Michael Heerdegen
2021-09-27  4:01     ` John Cummings
2021-09-27 23:29       ` Michael Heerdegen
2021-09-28 23:59         ` Michael Heerdegen
2021-09-29 16:44           ` John Cummings

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='0Ww226bb9COlx-Xd2_M8rgS9220VEQgQRuds0to9jte2TIRzsawQh9bHwsGHwYV_tDoA-1Xem_17CMQwZuMaGdn7x2x9nkbyKZyKtyvcOZs=@rootabega.net' \
    --to=john@rootabega.net \
    --cc=gnu-emacs-sources@gnu.org \
    --cc=help-gnu-emacs@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).