Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tree-sitter-cpp: add recipe #26172

Open
wants to merge 14 commits into
base: master
Choose a base branch
from

Conversation

perseoGI
Copy link
Contributor

@perseoGI perseoGI commented Dec 12, 2024

Summary

Changes to recipe: tree-sitter-cpp

Motivation

Closes #18487

Details

This PR aims to introduce C++ grammar for TreeSitter in CCI
Upstream repo: https://github.com/tree-sitter/tree-sitter-cpp
Right now we have available several TreeSitter grammars in CCI but C++ grammar was missing.

Adding it is pretty straight forward as this PR is completely based in existing tree-sitter-c grammar.
Since 0.23.2, TreeSitter grammars have an upstream CMakeFile which works like a charm with a minor patch which:

  • Removes the fPIC hardcoded to ON
  • Remove the find_program directive to tree-sitter-cli which will never be used in CCI due to src/parser.c grammar will always be available in sources.

Follow up of #18573


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[request] tree-sitter-cpp/v0.20.0
4 participants