suoto/hdl_checker

Added to Emacs lsp-mode

chrbirks opened this issue · 5 comments

chrbirks commented

Just a FYI. I added support for HDL Checker to the Emacs package lsp-mode: emacs-lsp/lsp-mode#1167
So it is now easy to just add the following to your Emacs config. Setting the server path is only required if the server binary is not in your PATH

(require 'use-package)
(setq lsp-vhdl-server-path "~/.local/bin/hdl_checker")
(custom-set-variables
 '(lsp-vhdl-server 'hdl-checker))
(use-package lsp-mode
	     :config (add-hook 'vhdl-mode-hook 'lsp))
suoto commented

That's great, thanks! :)

Would be nice to add this to the editors support section on the readme, but I'm not familiar with emacs at all. Maybe adding this bit and referencing emacs-lsp/lsp-mode enough?

(require 'use-package)
(setq lsp-vhdl-server-path "~/.local/bin/hdl_checker")
(custom-set-variables
 '(lsp-vhdl-server 'hdl-checker))
(use-package lsp-mode
	     :config (add-hook 'vhdl-mode-hook 'lsp))
chrbirks commented

Yes that should be enough. It requires that use-package is already installed but it's already pretty common.

chrbirks commented

Well the code was just reverted but it will be back again once I have fixed a problem :)

suoto commented

Ok, no worries. The readme has the section marked as "soon", will remove once the emacs-lsp is OK!

chrbirks commented

The problem has been fixed and merged into the lsp-mode package again.

  • 👍1
  • 🎉1