seagle0128 / doom-modeline

A fancy and fast mode-line inspired by minimalism design.
https://seagle0128.github.io/doom-modeline/
GNU General Public License v3.0
1.27k stars 157 forks source link

Error using with latest eglot / jsonrpc packages #689

Closed bradhowes closed 7 months ago

bradhowes commented 8 months ago

Thank you for the bug report

Bug description

I upgraded my packages and this brought down eglot v1.16 (built-in is 1.12.29). This installed an updated jsonrpc package v1.0.23. When starting a mode that uses eglot there now appears the following error:

Error running timer: (void-function jsonrpc--request-continuations)

This is due to the fact that that function no longer exists in the jsonrpc package.

I removed the eglot upgrade for now until doom-modeline is updated.

Steps to reproduce

Expected behavior

No error.

OS

MacOS

Emacs Version

29

Emacs Configurations

Custom

Error callstack

Debugger entered--Lisp error: (void-function jsonrpc--request-continuations)
  jsonrpc--request-continuations(#<eglot-lsp-server eglot-lsp-server-52c05d10>)
  doom-modeline-update-eglot()
  run-hooks(eglot-managed-mode-hook)
  eglot--maybe-activate-editing-mode()
  run-hooks(after-change-major-mode-hook)
  run-mode-hooks(python-mode-hook)
  python-mode()


### Anything else

Fix for now is to remove the `eglot` and `jsonrpc` upgrades for now until `doom-modeline` is updated.
bradhowes commented 8 months ago

Appears that jsonrpc-continuation-count would be the likely candidate to use instead. It works for me but the number that it shows make no sense -- string of repeated '100'...

EDIT: the count works OK. The string of repeated '100'... is due to something else I think (icon?) not related to this.

seagle0128 commented 8 months ago

Please make sure you are using the latest version of doom-modeline.

See https://github.com/seagle0128/doom-modeline/blob/master/doom-modeline-segments.el#L2072-L2076 .

bradhowes commented 7 months ago

Thanks for the response. I do not see anything more than 4.0.1 and that is what I've been using.

seagle0128 commented 7 months ago

Are you using the melpa-stable version? I've pushed a new tag: v4.1.0. Please try later.