GNU bug report logs - #74287
[PATCH] Rework history Isearch for Eshell

Previous Next

Full log

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

From: Juri Linkov <juri <at> linkov.net>
To: Jim Porter <jporterbugs <at> gmail.com>
Cc: 74287 <at> debbugs.gnu.org, Sean Whitton <spwhitton <at> spwhitton.name>,
 Pengji Zhang <me <at> pengjiz.com>, James Thomas <jimjoe <at> gmx.net>
Subject: Re: bug#74287: [PATCH] Rework history Isearch for Eshell
Date: Wed, 04 Dec 2024 20:56:29 +0200

> Like mentioned in the emacs-devel list, that's probably how *I'd* want
> a command history search implementation to use, but I also think that an
> Isearch interface makes a lot of sense as an option, depending on user
> preferences. As far as I can tell, we'd have to write different code for
> each of these styles (i.e. you can't just make Isearch use capf). Though as
> mentioned I'm not very familiar with Isearch's internals.

I confirm that the patch uses the right Isearch's internals
to implement the search through Eshell history.

(There is only one typo in the variable name
'eshell--history-isearch-messasge-overlay'.)

> So long as this doesn't prevent a capf-based implementation (and I'm
> guessing it wouldn't, since that would just be a new option), I think we
> could merge this patch. I also prefer to have Shell and Eshell work the
> same way unless there's some compelling reason why they should be
> different.

Having Shell and Eshell work the same way is a nice goal.
There is a patch that implements completion for Shell in bug#74694.
Then Eshell could have something like this later as well.

Previous Next

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