GNU bug report logs - #73722
30.0.91; error calling eshell/clear with no argument

Previous Next

Package: emacs;

Reported by: Christopher Howard <christopher <at> librehacker.com>

Date: Wed, 9 Oct 2024 17:34:02 UTC

Severity: normal

Found in version 30.0.91

Done: Jim Porter <jporterbugs <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Eli Zaretskii <eliz <at> gnu.org>
To: Jim Porter <jporterbugs <at> gmail.com>
Cc: christopher <at> librehacker.com, 73722 <at> debbugs.gnu.org
Subject: bug#73722: 30.0.91; error calling eshell/clear with no argument
Date: Wed, 16 Oct 2024 08:17:44 +0300
> Date: Tue, 15 Oct 2024 14:31:21 -0700
> Cc: christopher <at> librehacker.com, 73722 <at> debbugs.gnu.org
> From: Jim Porter <jporterbugs <at> gmail.com>
> 
> On 10/15/2024 11:04 AM, Eli Zaretskii wrote:
> >> Eli, does this look ok for the release branch? It fixes a regression
> >> from Emacs 29 and adds a pair of regression tests to prevent future
> >> breakage. (If and when this merges, I'll see about a more-thorough fix
> >> for the master branch too.)
> > 
> > If the problem is with empty body of progn, why does the fix have to
> > be so much less self-evident?  Can you fix it by protecting against
> > empty body of progn instead, so that the fix is less likely to cause
> > unintended consequences?
> 
> That's just the loudest part of the bug. Even though I've already fixed 
> that part on Emacs 31, things still don't work correctly there. If you 
> run the "clear" command in Eshell it clears the screen, but then prints 
> two new prompts on the same line together. However, this behavior is 
> pretty close to what "clear" did in Emacs 29: it prints two prompts (but 
> there's a newline between them).
> 
> If I fix the empty 'progn' case, I guess that would fix the regression 
> (aside from the missing newline, which I'm not sure how to fix). 
> However, "clear" still wouldn't work correctly.
> 
> What do you think would be better for Emacs 30? Fix "clear" so it works 
> properly, or just fix the new regression? (In the latter case, I can fix 
> it with a one-liner.)

At this late stage, I prefer the latter, yes.




This bug report was last modified 258 days ago.

Previous Next


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