abougouffa / minemacs

MinEmacs: an Emacs configuration framework for daily use
https://abougouffa.github.io/minemacs/
MIT License
154 stars 16 forks source link

[BUG] about tree-sitter #114

Closed donneyluck closed 1 year ago

donneyluck commented 1 year ago

Describe the bug tree-sitter not work
protobuf-ts-mode

Emacs version Run M-x emacs-version and put its output here: GNU Emacs 29.1 (build 2, x86_64-pc-linux-gnu, GTK+ Version 3.24.38, cairo version 1.17.8) of 2023-09-01

To Reproduce Steps to reproduce the behavior:

  1. add this in config.el
    (use-package protobuf-ts-mode
    :straight t
    :ensure t
    :mode ("\\.proto\\'" . protobuf-ts-mode))

    2.open one proto file

Screenshots image image this file already in protobuf-ts-mode but It seems that the syntax tree is not effective and there is no color keyword prompt.

image

I also tried other languages, such as c#, although the csharp-ts-mode & so files are already there, but the syntax is still not colored, and it still doesn’t seem to work.

image

Additional context When I open the proto file or switch protobuf-ts-mode, I will be asked to download libtree-sitter-protobuf.so* every time tree-sitter not work well M-x treesit-auto-install-all
It keeps prompting that protobuf has downloaded the .so file in time.

abougouffa commented 1 year ago

Thanks for the feedback,

I've noticed that the protobuf module keep asking for installation. However, I didn't try to dig deeper as I'm not working on any project with Protobuf. I will leave this issue open until I get time to dig deeper.

donneyluck commented 1 year ago

I tested it with c++ & c treesit and it works fine but there are some issues with csharp and protobuf I know you don't use these thanks for your time

abougouffa commented 1 year ago

I've just tried this and it seems buggy. After installing protobuf-ts-mode and opening a .proto file, treesit-auto keep asking to download the library (even though it is installed).

I will temporarily disable protobuf-ts-mode in treesit-auto until it gets fixed upstream.

@donneyluck Can you please open an issue in the treesit-auto upstream repo?

donneyluck commented 1 year ago

ok i will open an issue in the treesit-auto upstream repo