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.

Sign up for GitHub

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

Jump to bottom

revisit how we use ANTLR #548

Closed
cjchapman opened this issue Aug 10, 2018 · 2 comments · Fixed by #1367
Closed

revisit how we use ANTLR #548

cjchapman opened this issue Aug 10, 2018 · 2 comments · Fixed by #1367
Labels
c-tools CFF2 high priority important but not urgent makeotfexe

Comments

@cjchapman
Copy link
Contributor

Currently we have an old copy of ANTLR in the makeotf source tree. I think we should switch over to using the latest version, perhaps pulling it down as a git submodule or using a download from their homepage, rather than keeping a copy in our source tree.

Here's ANTLR on GitHub:
https://github.com/antlr

Here's the main ANTLR homepage:
http://www.antlr.org
@miguelsousa miguelsousa mentioned this issue Aug 10, 2018
Closed
19 tasks
@paravoid paravoid mentioned this issue Nov 5, 2018
Closed
@miguelsousa miguelsousa mentioned this issue Mar 25, 2019
Closed
@miguelsousa
Copy link
Member

Work on this should not start until #781 is resolved.

@cjchapman
Copy link
Contributor Author

See also the related issue #83.

@cjchapman cjchapman added the high priority important but not urgent label Oct 2, 2019
@iterumllc iterumllc mentioned this issue Jun 8, 2021
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c-tools CFF2 high priority important but not urgent makeotfexe
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Port of Feature File Parser to Antlr 4 iterumllc/afdko
3 participants
@khaledhosny @miguelsousa @cjchapman