GNU bug report logs - #58745
29.0.50; eglot managed-major-mode must be a list

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#58745: closed (29.0.50; eglot managed-major-mode must be a list)
Date: Thu, 27 Oct 2022 17:39:03 +0000

[Message part 1 (text/plain, inline)]

Your message dated Thu, 27 Oct 2022 20:37:54 +0300
with message-id <83a65hkw0d.fsf <at> gnu.org>
and subject line Re: bug#58745: 29.0.50; eglot managed-major-mode must be a list
has caused the debbugs.gnu.org bug report #58745,
regarding 29.0.50; eglot managed-major-mode must be a list
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
58745: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=58745
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Stephen Leake <stephen_leake <at> stephe-leake.org>
To: bug-gnu-emacs <at> gnu.org
Subject: 29.0.50; eglot managed-major-mode must be a list
Date: Sun, 23 Oct 2022 10:46:08 -0700

I'm calling `eglot' from lisp. The doc string says the first argument
`managed-major-mode' is an atom, not a list. But it is passed to
eglot--connect as the first argument, which is expected to be a list of
major modes. When called interactively, `managed-major-mode' is set by
(eglot--guess-contact t), which returns a list.

So when not interactive, `eglot' must convert `managed-major-mode' into
a list:

-------------------
diff --git a/lisp/progmodes/eglot.el b/lisp/progmodes/eglot.el
index 71001ba680..dee88db022 100644
--- a/lisp/progmodes/eglot.el
+++ b/lisp/progmodes/eglot.el
@@ -1078,6 +1078,8 @@ eglot
 
 INTERACTIVE is t if called interactively."
   (interactive (append (eglot--guess-contact t) '(t)))
+  (unless (listp managed-major-mode)
+    (setq managed-major-mode (list managed-major-mode)))
   (let* ((current-server (eglot-current-server))
          (live-p (and current-server (jsonrpc-running-p current-server))))
     (if (and live-p
------------------


-- 
-- Stephe
In GNU Emacs 29.0.50 (build 1, x86_64-w64-mingw32) of 2022-10-22 built
 on DESKTOP-G20DCG1
Repository revision: 9db7b11cf7fa5d6cf624a467f966a14b347e4429
Repository branch: master
Windowing system distributor 'Microsoft Corp.', version 10.0.19044
System Description: Microsoft Windows 10 Pro (v10.0.2009.19044.2130)

Configured using:
 'configure
 PKG_CONFIG_PATH=/mingw64/lib/pkgconfig:/mingw64/share/pkgconfig'

Configured features:
ACL DBUS GIF GMP GNUTLS HARFBUZZ JPEG JSON LCMS2 LIBXML2 MODULES NOTIFY
W32NOTIFY PDUMPER PNG RSVG SOUND SQLITE3 THREADS TIFF
TOOLKIT_SCROLL_BARS WEBP XPM ZLIB

Important settings:
  value of $LC_CTYPE: en_US.UTF-8
  value of $LANG: ENU
  locale-coding-system: cp1252

Major mode: ELisp/l

Minor modes in effect:
  bug-reference-prog-mode: t
  other-frame-window-mode: t
  display-time-mode: t
  delete-selection-mode: t
  outline-minor-mode: t
  icomplete-mode: t
  tooltip-mode: t
  global-eldoc-mode: t
  eldoc-mode: t
  show-paren-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  line-number-mode: t
  transient-mark-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t

Load-path shadows:
c:/Projects/org.emacs.ada-mode/prj hides c:/Projects/org.emacs.gpr-query/prj
c:/Projects/org.emacs.ada-mode/prj-eglot hides c:/Projects/org.emacs.gpr-query/prj-eglot
c:/Projects/org.emacs.ada-mode/autoloads hides c:/Projects/org.emacs.gpr-query/autoloads
c:/Projects/org.emacs.ada-mode/autoloads hides c:/Projects/org.emacs.gnat-compiler/autoloads
c:/Projects/org.emacs.ada-mode/prj-eglot hides c:/Projects/org.emacs.wisi/prj-eglot
c:/Projects/org.emacs.ada-mode/autoloads hides c:/Projects/org.emacs.wisi/autoloads
c:/Projects/org.emacs.ada-mode/prj hides c:/Projects/org.wisitoken.grammar_mode/prj
c:/Projects/org.emacs.ada-mode/autoloads hides c:/Projects/org.wisitoken.grammar_mode/autoloads
c:/Projects/org.emacs.ada-mode/ada-ref-man hides c:/home/stephe/.emacs.d/elpa/ada-ref-man-2020.1.0.20201129.190419/ada-ref-man
c:/Projects/mmm-mode/mmm-vars hides c:/home/stephe/.emacs.d/elpa/mmm-mode-0.5.8.0.20221018.153958/mmm-vars
c:/Projects/mmm-mode/mmm-utils hides c:/home/stephe/.emacs.d/elpa/mmm-mode-0.5.8.0.20221018.153958/mmm-utils
c:/Projects/mmm-mode/mmm-univ hides c:/home/stephe/.emacs.d/elpa/mmm-mode-0.5.8.0.20221018.153958/mmm-univ
c:/Projects/mmm-mode/mmm-sample hides c:/home/stephe/.emacs.d/elpa/mmm-mode-0.5.8.0.20221018.153958/mmm-sample
c:/Projects/mmm-mode/mmm-rpm hides c:/home/stephe/.emacs.d/elpa/mmm-mode-0.5.8.0.20221018.153958/mmm-rpm
c:/Projects/mmm-mode/mmm-region hides c:/home/stephe/.emacs.d/elpa/mmm-mode-0.5.8.0.20221018.153958/mmm-region
c:/Projects/mmm-mode/mmm-noweb hides c:/home/stephe/.emacs.d/elpa/mmm-mode-0.5.8.0.20221018.153958/mmm-noweb
c:/Projects/mmm-mode/mmm-myghty hides c:/home/stephe/.emacs.d/elpa/mmm-mode-0.5.8.0.20221018.153958/mmm-myghty
c:/Projects/mmm-mode/mmm-mode hides c:/home/stephe/.emacs.d/elpa/mmm-mode-0.5.8.0.20221018.153958/mmm-mode
c:/Projects/mmm-mode/mmm-mason hides c:/home/stephe/.emacs.d/elpa/mmm-mode-0.5.8.0.20221018.153958/mmm-mason
c:/Projects/mmm-mode/mmm-erb hides c:/home/stephe/.emacs.d/elpa/mmm-mode-0.5.8.0.20221018.153958/mmm-erb
c:/Projects/mmm-mode/mmm-defaults hides c:/home/stephe/.emacs.d/elpa/mmm-mode-0.5.8.0.20221018.153958/mmm-defaults
c:/Projects/mmm-mode/mmm-cweb hides c:/home/stephe/.emacs.d/elpa/mmm-mode-0.5.8.0.20221018.153958/mmm-cweb
c:/Projects/mmm-mode/mmm-compat hides c:/home/stephe/.emacs.d/elpa/mmm-mode-0.5.8.0.20221018.153958/mmm-compat
c:/Projects/mmm-mode/mmm-cmds hides c:/home/stephe/.emacs.d/elpa/mmm-mode-0.5.8.0.20221018.153958/mmm-cmds
c:/Projects/mmm-mode/mmm-class hides c:/home/stephe/.emacs.d/elpa/mmm-mode-0.5.8.0.20221018.153958/mmm-class
c:/Projects/mmm-mode/mmm-auto hides c:/home/stephe/.emacs.d/elpa/mmm-mode-0.5.8.0.20221018.153958/mmm-auto

Features:
(shadow sort mail-extr emacsbug pulse color vc bug-reference help-fns
radix-tree cl-print misearch multi-isearch copyright cus-start edebug
ada-eglot ada-mode ada-imenu align ada-skel wisi-skel skeleton
ada_annex_p-process wisi-process-parse ada-indent-user-options ada-core
gnat-compiler vc-git diff-mode easy-mmode vc-dispatcher org-element
avl-tree generator ol-eww eww xdg url-queue mm-url ol-rmail ol-mhe
ol-irc ol-info ol-gnus nnselect gnus-art mm-uu mml2015 mm-view mml-smime
smime gnutls dig gnus-sum shr pixel-fill kinsoku url-file svg dom
gnus-group gnus-undo gnus-start gnus-dbus dbus xml gnus-cloud nnimap
nnmail mail-source utf7 nnoo parse-time gnus-spec gnus-int gnus-range
message sendmail yank-media puny rfc822 mml mml-sec epa derived
mm-decode mm-bodies mm-encode mail-parse rfc2231 rfc2047 rfc2045
ietf-drums mailabbrev gmm-utils mailheader gnus-win gnus nnheader
gnus-util mail-utils range mm-util mail-prsvr wid-edit ol-docview
doc-view jka-compr image-mode exif ol-bibtex ol-bbdb ol-w3m ol-doi
org-link-doi org-mode-keys org ob ob-tangle ob-ref ob-lob ob-table
ob-exp org-macro org-footnote org-src ob-comint org-pcomplete pcomplete
org-list org-faces org-entities org-version ob-emacs-lisp ob-core
ob-eval org-table oc-basic bibtex iso8601 time-date ol rx org-keys oc
org-compat advice org-macs org-loaddefs format-spec cal-menu calendar
cal-loaddefs wisi-prj wisi wisi-fringe wisi-parse-common semantic/lex
semantic/fw mode-local find-file eglot array filenotify jsonrpc ert pp
debug backtrace find-func flymake-proc flymake warnings pcase imenu
mmm-auto mmm-vars mmm-utils mmm-compat dvc-autoloads dvc-unified ffap
thingatpt dvc-revlist dvc-fileinfo dvc-core dvc-buffers dvc-ui
dvc-register dvc-utils ewoc dvc-emacs dvc-defs autoloads
other-frame-window grep time delsel cus-load stephe-theme noutline
outline icons path-iterator cl-extra help-mode whitespace dired-x
dired-aux dired dired-loaddefs compile text-property-search comint
ansi-osc ansi-color uniquify-files icomplete xref project ring
ada-ref-man-autoloads bbdb-autoloads info debbugs-autoloads
gnu-elpa-keyring-update-autoloads epg rfc6068 epg-config
gnu-elpa-keyring-update helm-autoloads helm-core-autoloads
async-autoloads mmm-mode-autoloads other-frame-window-autoloads
popup-autoloads uniquify-files-autoloads package browse-url url
url-proxy url-privacy url-expand url-methods url-history url-cookie
generate-lisp-file url-domsuf url-util mailcap url-handlers url-parse
auth-source cl-seq eieio eieio-core cl-macs password-cache json subr-x
map byte-opt gv bytecomp byte-compile cconv url-vars cl-loaddefs cl-lib
rmc iso-transl tooltip eldoc paren electric uniquify ediff-hook vc-hooks
lisp-float-type elisp-mode mwheel dos-w32 ls-lisp disp-table
term/w32-win w32-win w32-vars term/common-win tool-bar dnd fontset image
regexp-opt fringe tabulated-list replace newcomment text-mode lisp-mode
prog-mode register page tab-bar menu-bar rfn-eshadow isearch easymenu
timer select scroll-bar mouse jit-lock font-lock syntax font-core
term/tty-colors frame minibuffer nadvice seq simple cl-generic
indonesian philippine cham georgian utf-8-lang misc-lang vietnamese
tibetan thai tai-viet lao korean japanese eucjp-ms cp51932 hebrew greek
romanian slovak czech european ethiopic indian cyrillic chinese
composite emoji-zwj charscript charprop case-table epa-hook
jka-cmpr-hook help abbrev obarray oclosure cl-preloaded button loaddefs
theme-loaddefs faces cus-face macroexp files window text-properties
overlay sha1 md5 base64 format env code-pages mule custom widget keymap
hashtable-print-readable backquote threads w32notify dbusbind w32 lcms2
multi-tty make-network-process emacs)

Memory information:
((conses 16 283572 42382)
 (symbols 48 29015 0)
 (strings 32 97057 4158)
 (string-bytes 1 2980479)
 (vectors 16 51760)
 (vector-slots 8 619438 29596)
 (floats 8 411 501)
 (intervals 56 3878 22)
 (buffers 1000 28))

[Message part 3 (message/rfc822, inline)]

From: Eli Zaretskii <eliz <at> gnu.org>
To: João Távora <joaotavora <at> gmail.com>
Cc: 58745-done <at> debbugs.gnu.org, stephen_leake <at> stephe-leake.org
Subject: Re: bug#58745: 29.0.50; eglot managed-major-mode must be a list
Date: Thu, 27 Oct 2022 20:37:54 +0300

> From: João Távora <joaotavora <at> gmail.com>
> Date: Thu, 27 Oct 2022 18:27:14 +0100
> Cc: Stephen Leake <stephen_leake <at> stephe-leake.org>, 58745 <at> debbugs.gnu.org
> 
> Yes, this has been discussed on emacs-devel and an equivalent patch
> pushed.
> 
> So the bug can be closed.

Done.

Previous Next

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