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

[build] mkindex.py/grammars.json should be updated, and phased out. #3420

Closed
kaby76 opened this issue May 14, 2023 · 0 comments · Fixed by #3421
Closed

[build] mkindex.py/grammars.json should be updated, and phased out. #3420

kaby76 opened this issue May 14, 2023 · 0 comments · Fixed by #3421

Comments

@kaby76
Copy link
Contributor

kaby76 commented May 14, 2023

lab.antlr.org uses grammars.json in this repo. Unfortunately, lab.antlr.org uses the "Antlr interpreter", and any grammars that have actions (e.g., "semantic predicates") will not work. Folks starting out with Antlr will report bugs here and elsewhere for grammars that cannot run. It does not make sense to even offer in lab.antlr.org these grammars. The grammars.json file should be updated to remove all grammars that contain actions.

lab.antlr.org is minimally supported software at this point. antlr/antlr4-lab#69 (comment). The need to build the grammars.json file should be dropped.

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

Successfully merging a pull request may close this issue.

2 participants