GNU bug report logs - #70039
30.0.50; list-packages install: Invalid read syntax: "#"

Previous Next

Package: emacs;

Reported by: Michael Heerdegen <michael_heerdegen <at> web.de>

Date: Wed, 27 Mar 2024 21:11:01 UTC

Severity: normal

Found in version 30.0.50

Done: Stefan Kangas <stefankangas <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 70039 in the body.
You can then email your comments to 70039 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 monnier <at> iro.umontreal.ca, bug-gnu-emacs <at> gnu.org:
bug#70039; Package emacs. (Wed, 27 Mar 2024 21:11:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Michael Heerdegen <michael_heerdegen <at> web.de>:
New bug report received and forwarded. Copy sent to monnier <at> iro.umontreal.ca, bug-gnu-emacs <at> gnu.org. (Wed, 27 Mar 2024 21:11:02 GMT) Full text and rfc822 format available.

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

From: Michael Heerdegen <michael_heerdegen <at> web.de>
To: bug-gnu-emacs <at> gnu.org
Subject: 30.0.50; list-packages install: Invalid read syntax: "#"
Date: Wed, 27 Mar 2024 22:10:40 +0100
[Message part 1 (text/plain, inline)]
Hello,

got this new error today when trying to install a package with M-x
list-packages.

When I revert this commit:

  7e32e8392ab "Fix recent test regressions"
  (Stefan Monnier <monnier <at> iro.umontreal.ca> 2024-03-22)

the problem disappears.  This commit causes a destruction of my custom
file when it gets pp-filled, by adding newlines into #N# syntaxes, like
#1# => #1\n#.

Attached is a test file.  If you (pp-fill (point-min) (point-max))
you will see this effect happen near the end of that file.


TIA,

Michael.


[pp-fill-test.el (application/emacs-lisp, attachment)]

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#70039; Package emacs. (Thu, 28 Mar 2024 19:35:01 GMT) Full text and rfc822 format available.

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

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Michael Heerdegen <michael_heerdegen <at> web.de>
Cc: 70039 <at> debbugs.gnu.org
Subject: Re: bug#70039: 30.0.50; list-packages install: Invalid read syntax:
 "#"
Date: Thu, 28 Mar 2024 15:34:15 -0400
> got this new error today when trying to install a package with M-x
> list-packages.
>
> When I revert this commit:
>
>   7e32e8392ab "Fix recent test regressions"
>   (Stefan Monnier <monnier <at> iro.umontreal.ca> 2024-03-22)
>
> the problem disappears.  This commit causes a destruction of my custom
> file when it gets pp-filled, by adding newlines into #N# syntaxes, like
> #1# => #1\n#.

I pushed a patch to `master` which should fix this problem, but ... this
is nasty.  I was thinking "how quickly one forgets how hard it is to
parse backward" (thinking back to the time where I worked on
`comment-backward`), but while testing I came across a similar problem
that occurs because of the (forward) `scan-sexps` behavior, so it's even
worse than that.


        Stefan





Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#70039; Package emacs. (Fri, 29 Mar 2024 11:51:02 GMT) Full text and rfc822 format available.

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

From: Michael Heerdegen <michael_heerdegen <at> web.de>
To: Stefan Monnier <monnier <at> iro.umontreal.ca>
Cc: 70039 <at> debbugs.gnu.org
Subject: Re: bug#70039: 30.0.50; list-packages install: Invalid read syntax:
 "#"
Date: Fri, 29 Mar 2024 12:51:07 +0100
Stefan Monnier <monnier <at> iro.umontreal.ca> writes:

> > the problem disappears.  This commit causes a destruction of my custom
> > file when it gets pp-filled, by adding newlines into #N# syntaxes, like
> > #1# => #1\n#.
>
> I pushed a patch to `master` which should fix this problem

Works for me, thanks.

> , but ... this is nasty.  I was thinking "how quickly one forgets how
> hard it is to parse backward" (thinking back to the time where I
> worked on `comment-backward`), but while testing I came across a
> similar problem that occurs because of the (forward) `scan-sexps`
> behavior, so it's even worse than that.

We call this a "wasps' nest" in Germany (of course using German words),
and I make this experience often.

Anyway, if you come along fixing Bug#24542, that would be - it would be
one more wasp less...


Michael.




Reply sent to Stefan Kangas <stefankangas <at> gmail.com>:
You have taken responsibility. (Sat, 01 Mar 2025 03:02:02 GMT) Full text and rfc822 format available.

Notification sent to Michael Heerdegen <michael_heerdegen <at> web.de>:
bug acknowledged by developer. (Sat, 01 Mar 2025 03:02:02 GMT) Full text and rfc822 format available.

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

From: Stefan Kangas <stefankangas <at> gmail.com>
To: Michael Heerdegen <michael_heerdegen <at> web.de>
Cc: 70039-done <at> debbugs.gnu.org, Stefan Monnier <monnier <at> iro.umontreal.ca>
Subject: Re: bug#70039: 30.0.50;
 list-packages install: Invalid read syntax: "#"
Date: Fri, 28 Feb 2025 19:01:07 -0800
Michael Heerdegen <michael_heerdegen <at> web.de> writes:

> Stefan Monnier <monnier <at> iro.umontreal.ca> writes:
>
>> > the problem disappears.  This commit causes a destruction of my custom
>> > file when it gets pp-filled, by adding newlines into #N# syntaxes, like
>> > #1# => #1\n#.
>>
>> I pushed a patch to `master` which should fix this problem
>
> Works for me, thanks.

I'm therefore closing this bug report.




bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Sat, 29 Mar 2025 11:24:24 GMT) Full text and rfc822 format available.

This bug report was last modified 85 days ago.

Previous Next


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