-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Eager macro-expansion failure #81
Comments
I met the same error and I'm also on Emacs 29. It seems some upstream changes broke some function in @thoradam Which commit of Emacs 29 are you using (and has this problem)? And which commit is good? I could do a git bisect to find which commit is responsible for this problem. |
@VitalyAnkh I installed Emacs from the Homebrew emacs-plus formula:
Unfortunately this is my first time using Emacs with lsp-metals so I don't have a working commit for reference. |
Treeview is broken after breaking the change in treemacs. It should be disabled until this is fixed. |
what is the status of this? what is the recommended work around until fixed (one which lets you continue to use metals)? |
This patch for the treemacs package works for me: diff --git a/src/elisp/treemacs-treelib.el b/src/elisp/treemacs-treelib.el
index d30c8c6..9825fbf 100644
--- a/src/elisp/treemacs-treelib.el
+++ b/src/elisp/treemacs-treelib.el
@@ -1018,7 +1018,7 @@ ARG: Prefix Arg"
:post-close-action
(treemacs-on-collapse (treemacs-button-get btn :path))))
-(cl-defmacro treemacs-initialize
+(cl-defmacro treemacs-initialize-ext
(extension
&key
(with-expand-depth 0) It seems that lsp-metals wants to call the other |
Does it stop work completely? For me lsp-metals works fine after this error happens. |
It stopped working in the latest Emacs (snapshot) due to the commit emacs-mirror/emacs@80cf13a eager expansions is now error not warning. Related issue: emacs-lsp/lsp-mode#3738 |
Temporary fix for emacs-lsp#81 `Eager macro-expansion failure: %S" (wrong-number-of-arguments (1 . 1) 0` This error is occurring even after `(require treemacs-extensions)` fix. Just passing an argument works.
When I open a Scala file and lsp-metals-treeview initialises I get an Emacs error:
"Eager macro-expansion failure: %S" (wrong-number-of-arguments (1 . 1) 0
I'm using Emacs 29. Backtrace:
The text was updated successfully, but these errors were encountered: