fix(flycheck): wrap languagetool checker definition in eval+backquote

flycheck's `flycheck-define-checker' macro requires the `:command'
executable to be a string literal at macro-expansion time -- it does
`(stringp (car command))' and errors otherwise.  The previous
`(eval (expand-file-name ...))' form (commit ad3b7617, the
externalization fix) put a `(eval FORM)' wrapper in the executable
position, which flycheck rejected at load:

  Error (use-package): recentf/:config: Command executable for
  syntax checker languagetool must be a string:
  (eval (expand-file-name "scripts/languagetool-flycheck"
                          user-emacs-directory))

`(eval FORM)' is only valid for SUBSEQUENT command-list elements
(arguments), not the executable.

Wrap the entire `flycheck-define-checker' invocation in `eval' +
backquote so the expanded path is spliced in as a string literal
before the macro sees it.  The hardcoded `~/.emacs.d/...' path is
gone for the same reason the original externalization wanted it
gone: survives a non-standard `user-emacs-directory'.

0 files changed, 0 insertions, 0 deletions