GNU bug report logs -
#61849
29.0.60; Unable to use treesit-install-language-grammar because repo doesn't have parser.c
Previous Next
Full log
Message #38 received at 61849 <at> debbugs.gnu.org (full text, mbox):
On 20/03/2023 21:28, Eli Zaretskii wrote:
>> From: Misha Zharov<mishazharov1 <at> gmail.com>
>> Date: Sun, 19 Mar 2023 22:14:44 -0700
>> Cc:casouri <at> gmail.com,61849 <at> debbugs.gnu.org
>>
>>> Please chime in to try to convince them to go back to including the
>>> generated parser files.
>> Thanks for the update. I've now done this, but I think that it will be
>> difficult to make sure that community sticks to this standard of keeping
>> the autogenerated files in the repo. Their arguments do have some
>> merit regarding the difficulty of resolving merge conflicts and overall
>> maintenance burden. At the same time, having the autogenerated file
>> present is a nice QOL feature for users because it's easily reproducible.
> Bummer:
>
> https://github.com/DerekStride/tree-sitter-sql/issues/120#issuecomment-1476609242
Suppose the repositories remove the generated grammar files. What would
be sufficient for us to regenerate them?
tree-sitter-sql apparently uses the tree-sitter-cli program. Would
having it on the user's system suffice?
For a lot of developers NPM will already be installed. If the only
remaining step will be 'npm install -g tree-sitter-cli', or dropping one
of the pre-built binaries from
https://github.com/tree-sitter/tree-sitter/releases into a directory on
PATH, I think they could manage to do it once, to be used in all
grammars which don't keep generated files in the repo.
This bug report was last modified 84 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.