emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: master 44fcd37a486 08/11: Add more c-ts-mode indent tests


From: Yuan Fu
Subject: Re: master 44fcd37a486 08/11: Add more c-ts-mode indent tests
Date: Fri, 6 Dec 2024 17:04:42 -0800


> On Dec 5, 2024, at 11:30 PM, Yuan Fu <casouri@gmail.com> wrote:
> 
> 
> 
>> On Dec 4, 2024, at 11:37 PM, Michael Albinus <michael.albinus@gmx.de> wrote:
>> 
>> Yuan Fu <casouri@gmail.com> writes:
>> 
>> Hi Yuan,
>> 
>>> This is baffling. I assume the pipeline build the latest
>>> tree-sitter-c/cpp grammar right? I just updated my grammar and
>>> tree-sitter lib to the latest and the tests still pass on my
>>> machine. Is tree-sitter version also the latest?
>> 
>> See test/infra/Dockerfile.emba. The emacs-tree-sitter container image is
>> created prior every run, the treesitter grammars mentioned there are
>> installed.
> 
> Ah, so the image uses Debian bookworm, which AFAIK is the current stable 
> release, so it probably doesn’t have the latest tree-sitter library. Let me 
> try to run the pipeline with tree-sitter built from upstream.
> 
> Yuan

I didn’t run the pipeline built from upstream, but I tried running the tests 
locally with the tree-sitter library used in the pipeline. The pipeline uses 
tree-sitter provided by bookworm, which is v0.20.7, a pretty old version. And 
indeed, with that version of tree-sitter I can see the test failure.

Andrea, which version of  tree-sitter do you have on your test bench?

Since tree-sitter is still relatively unstable, maybe we want to use upstream 
tree-sitter for the test suite?

Yuan




reply via email to

[Prev in Thread] Current Thread [Next in Thread]