GNU bug report logs - #48770
Command `eshell-command` raises error when `eshell-hist-ignoredups` is 'erase

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 48770 in the body.
You can then email your comments to 48770 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: Okam <okamsn <at> protonmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: Command `eshell-command` raises error when `eshell-hist-ignoredups`
 is 'erase
Date: Tue, 01 Jun 2021 02:21:46 +0000

Hello,

This raises the "Ring empty" error because there is no history in
`eshell-history-ring` when I use this command. This command doesn't seem
to be aware of the input history of other Eshell buffers.

Thank you.

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Okam <okamsn <at> protonmail.com>
Cc: 48770 <at> debbugs.gnu.org
Subject: Re: bug#48770: Command `eshell-command` raises error when
 `eshell-hist-ignoredups` is 'erase
Date: Tue, 01 Jun 2021 17:48:37 +0200

Okam <okamsn <at> protonmail.com> writes:

> This raises the "Ring empty" error because there is no history in
> `eshell-history-ring` when I use this command. This command doesn't seem
> to be aware of the input history of other Eshell buffers.

I'm not quite sure I understand.  I tried `M-x eshell-command' with that
setting, and I got no error in Emacs 28.

What Emacs version is this with?

Do you have a step-by-step recipe to reproduce the problem, starting
from "emacs -Q"?

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

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

From: Okam <okamsn <at> protonmail.com>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 48770 <at> debbugs.gnu.org
Subject: Re: bug#48770: Command `eshell-command` raises error when
 `eshell-hist-ignoredups` is 'erase
Date: Wed, 02 Jun 2021 21:26:08 +0000

On 6/1/21 11:48 AM, Lars Ingebrigtsen wrote:
 > What Emacs version is this with?

This is from the master branch with with repository version
0c7f1e2e42d6bf9f95e88c02d4e1ed9cb40693d8, as determined by
`emacs-repository-version`.

I also tried pulling and compiling from the master branch with
repository version 09081ca119c3debcc72159a46d3bafabfd7519bb.  The error
is still raised.

This is with native compilation enabled, if that changes things.

 > Do you have a step-by-step recipe to reproduce the problem, starting
 > from "emacs -Q"?

1. Start Emacs with `emacs -Q`
2. Evaluate `(setq eshell-hist-ignoredups 'erase)`
3. Run `M-x eshell-command`
4. Run a command, such as `ls`.
5. The error "ring-remove: Ring empty" is signaled.
6. The directory listing is not created.

The value of `eshell-history-ring` during this is

(0 0 .
    [nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil
nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil
nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil
nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil
nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil
nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil
nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil nil
nil nil nil])

Does any of this information help?

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Okam <okamsn <at> protonmail.com>
Cc: 48770 <at> debbugs.gnu.org
Subject: Re: bug#48770: Command `eshell-command` raises error when
 `eshell-hist-ignoredups` is 'erase
Date: Thu, 03 Jun 2021 09:51:03 +0200

Okam <okamsn <at> protonmail.com> writes:

> 1. Start Emacs with `emacs -Q`
> 2. Evaluate `(setq eshell-hist-ignoredups 'erase)`
> 3. Run `M-x eshell-command`
> 4. Run a command, such as `ls`.
> 5. The error "ring-remove: Ring empty" is signaled.
> 6. The directory listing is not created.

Thanks for the clear recipe.  With that, I was able to reproduce the
problem, and I've now pushed a fix to Emacs 28.

-- 
(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.