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

support --external-tokens like menhir(ocamlyacc) #7380

Closed
vicuna opened this issue Oct 5, 2016 · 5 comments
Closed

support --external-tokens like menhir(ocamlyacc) #7380

vicuna opened this issue Oct 5, 2016 · 5 comments

Comments

@vicuna
Copy link

vicuna commented Oct 5, 2016

Original bug ID: 7380
Reporter: @bobzhang
Status: feedback (set by @shindere on 2017-02-24T15:24:19Z)
Resolution: open
Priority: normal
Severity: feature
Category: tools (ocaml{lex,yacc,dep,debug,...})

Bug description

currently the ocamlyacc will impose lexer depends on parser.
This is not necessary, and sometimes you have more tokens than the parser need

@vicuna
Copy link
Author

vicuna commented Oct 5, 2016

Comment author: @gasche

I don't understand why people are still using ocamlyacc in this day and age, Menhir is superior in many different respects and ocamlyacc should be deprecated. I don't want to invest work in improving ocamlyacc myself. Of course, if you find people willing to write a patch and others to review it, feel free to.

I would be more interested in hearing about the issues that make people prefer ocamlyacc over Menhir. I know that, for existing parsers, transitioning is not so easy because of the different interface to access source locations. Are there other issues that should be considered?

@vicuna
Copy link
Author

vicuna commented Oct 5, 2016

Comment author: @bobzhang

@gasche, I understand menhir is superior to ocamlyacc.

The main reason is dependencies. In commercial environment(does not apply to academy or startups), every piece of software has to be carefully reviewed by lawyers, this may take several months if lucky. In most cases, developers won't bother, just use the software already boss approved (here the official ocaml distribution: ocamlyacc)

I am happy to write patches for it.

@vicuna
Copy link
Author

vicuna commented Feb 24, 2017

Comment author: @shindere

Why would a project that needs such a feature use ocamlyacc rather than menhir, actually?
Given the legacy status of ocamlyacc and the discussions around its replacement by Menhir even for the OCaml parser itself it seems unlikely to me that such a feature will be implemented in Menhir one day so I'm tempted to say this won't be fixed and close this issue.

@github-actions
Copy link

github-actions bot commented May 9, 2020

This issue has been open one year with no activity. Consequently, it is being marked with the "stale" label. What this means is that the issue will be automatically closed in 30 days unless more comments are added or the "stale" label is removed. Comments that provide new information on the issue are especially welcome: is it still reproducible? did it appear in other contexts? how critical is it? etc.

@github-actions github-actions bot added the Stale label May 9, 2020
@xavierleroy
Copy link
Contributor

We're in 2020, Menhir is the recommended parser generator for OCaml, ocamlyacc is legacy software and will not be chnaged except to fix serious bugs.

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

No branches or pull requests

2 participants