all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#52794: 28.0.90; Eshell: error while completing
@ 2021-12-25 17:50 Óscar Fuentes
  2021-12-27 14:50 ` Óscar Fuentes
  2021-12-27 14:52 ` Lars Ingebrigtsen
  0 siblings, 2 replies; 4+ messages in thread
From: Óscar Fuentes @ 2021-12-25 17:50 UTC (permalink / raw)
  To: 52794


emacs -Q

M-x eshell

Go to a directory with two or more files with the same prefix. The Emacs
build directory, for instance. Let's suppose the prefix is `c'.

mv c* foo[TAB]

Debugger entered--Lisp error: (wrong-type-argument stringp ("config.log" "config.log~" "config.status"))
  string-match("\\.\\.\\.+/" ("config.log" "config.log~" "config.status"))
  #f(compiled-function (arg) #<bytecode 0x18cd73f57cbbd741>)((eshell-extended-glob (eshell-convert (concat "c" "*"))))
  mapcar(#f(compiled-function (arg) #<bytecode 0x18cd73f57cbbd741>) ("mv" (eshell-extended-glob (eshell-convert (concat "c" "*"))) "l"))
  eshell-complete-parse-arguments()
  pcomplete-parse-arguments(nil)
  pcomplete-completions()
  pcomplete-completions-at-point()
  completion--capf-wrapper(pcomplete-completions-at-point all)
  run-hook-wrapped(completion--capf-wrapper pcomplete-completions-at-point all)
  completion-at-point()
  funcall-interactively(completion-at-point)
  call-interactively(completion-at-point nil nil)
  command-execute(completion-at-point)


The patch below fixes the problem, which if my analysis is correct, was
introduced by:

commit 9224a863192b1317ef307bcc76abfdfbad73b796
Author: Michalis V <mvar.40k@gmail.com>
Date:   Fri Aug 27 18:46:04 2021 +0200

    Fix completion of extended "..." syntax in eshell
    
    * lisp/eshell/em-cmpl.el (eshell-complete-parse-arguments): Expand
    "..." (bug#19626).


emacs/lisp/eshell/em-cmpl.el
@@ -380,7 +380,7 @@
                       (setq val (number-to-string val)))
                      ;; expand .../ etc that only eshell understands to
                      ;; standard ../../
-                     ((string-match "\\.\\.\\.+/" val)
+                     ((and (stringp val) (string-match "\\.\\.\\.+/" val))
                       (setq val (eshell-expand-multiple-dots val))))
                (or val "")))
 	   args)

The problem is that `val' is a list when more than one file matches the
glob, and then `string-match' errors-out.

Ok to commit in emacs-28 ?





^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#52794: 28.0.90; Eshell: error while completing
  2021-12-25 17:50 bug#52794: 28.0.90; Eshell: error while completing Óscar Fuentes
@ 2021-12-27 14:50 ` Óscar Fuentes
  2021-12-27 14:52 ` Lars Ingebrigtsen
  1 sibling, 0 replies; 4+ messages in thread
From: Óscar Fuentes @ 2021-12-27 14:50 UTC (permalink / raw)
  To: 52794

Ping.

> Ok to commit to emacs-28 ?





^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#52794: 28.0.90; Eshell: error while completing
  2021-12-25 17:50 bug#52794: 28.0.90; Eshell: error while completing Óscar Fuentes
  2021-12-27 14:50 ` Óscar Fuentes
@ 2021-12-27 14:52 ` Lars Ingebrigtsen
  2021-12-27 15:07   ` Óscar Fuentes
  1 sibling, 1 reply; 4+ messages in thread
From: Lars Ingebrigtsen @ 2021-12-27 14:52 UTC (permalink / raw)
  To: Óscar Fuentes; +Cc: 52794

Óscar Fuentes <ofv@wanadoo.es> writes:

> The problem is that `val' is a list when more than one file matches the
> glob, and then `string-match' errors-out.
>
> Ok to commit in emacs-28 ?

Yup.

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





^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#52794: 28.0.90; Eshell: error while completing
  2021-12-27 14:52 ` Lars Ingebrigtsen
@ 2021-12-27 15:07   ` Óscar Fuentes
  0 siblings, 0 replies; 4+ messages in thread
From: Óscar Fuentes @ 2021-12-27 15:07 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 52794-done

Lars Ingebrigtsen <larsi@gnus.org> writes:

>> Ok to commit in emacs-28 ?
>
> Yup.

Thanks. Pushed as ea65de757766fa4cb5a090012e2f6a0aeef0b9f8.





^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-12-27 15:07 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-12-25 17:50 bug#52794: 28.0.90; Eshell: error while completing Óscar Fuentes
2021-12-27 14:50 ` Óscar Fuentes
2021-12-27 14:52 ` Lars Ingebrigtsen
2021-12-27 15:07   ` Óscar Fuentes

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.