emenel/doomemacs
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

fix(eshell): eshell-did-you-mean: sequencep 771 error

Browse source 
Also removes obsolete hack for eshell-did-you-mean (see
xuchunyang/eshell-did-you-mean#2).

Ref: xuchunyang/eshell-did-you-mean#2
This commit is contained in:
Henrik Lissner 2024-08-21 14:53:23 -04:00
parent 122e3732f7
commit dbcd30820b
No known key found for this signature in database
GPG key ID: B60957CA074D39A3
1 changed files with 11 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

18
modules/term/eshell/config.el
View file

@ -226,13 +226,17 @@ Emacs versions < 29."
(use-package! eshell-did-you-mean (use-package! eshell-did-you-mean
:after esh-mode ; Specifically esh-mode, not eshell :after esh-mode ; Specifically esh-mode, not eshell
:config :config (eshell-did-you-mean-setup)
(eshell-did-you-mean-setup)
;; HACK There is a known issue with `eshell-did-you-mean' where it does not ;; HACK: `pcomplete-completions' returns a function, but
;; work on first invocation, so we invoke it once manually by setting the ;; `eshell-did-you-mean--get-all-commands' unconditionally expects it to
;; last command and then calling the output filter. ;; return a list of strings, causing wrong-type-arg errors in many cases.
(setq eshell-last-command-name "catt") ;; `all-completions' handles all these cases.
(eshell-did-you-mean-output-filter "catt: command not found")) (defadvice! +eshell--fix-eshell-did-you-mean-a (&rest _)
:override #'eshell-did-you-mean--get-all-commands
(unless eshell-did-you-mean--all-commands
(setq eshell-did-you-mean--all-commands
(all-completions "" (pcomplete-completions))))))
(use-package eshell-syntax-highlighting (use-package eshell-syntax-highlighting