-
Notifications
You must be signed in to change notification settings - Fork 12
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
Replace camlp4 with ppx #40
Comments
It should not be terribly hard to implement this in a minimal way. Macaque was intentionally designed to avoid extending the OCaml grammar, it almost exclusively relies on There is one (optional) part of Macaque that changes the OCaml grammar, it is pa_banana that allows chaining of structured row data access, using the However, I don't have time to implement this in the short-term. I would be happy to accept contributions and provide code review and recomendations, though; it is in that spirit that I tried to provide a detailed forward path above. Finally, as I have already discussed with several Macaque users, I would be interested in eventually transitioning from the set-comprehension syntax to something closer to the actual SQL syntax (to avoid having to do big syntactic brainstorming each time we want to add a new SQL feature; just reuse the damn thing). @Drup started working on that but nothing conclusive emerged so far, and I think his work on this is stalled. @hnrgrgr has developped a SQL parser in OCaml that I think we could reuse. Developing a ppx extension means changing the syntax, so it could be a good time to think about also providing a SQL-based one -- but there is no hurry on this one. |
Has there been any motion on this? If not, I'm currently looking at a project where I may be needing something like this, in which case I'd be happy to do the footwork. |
No, I don't think anyone has worked on it since my last message above, sorry. If you are willing to implement something, I would be happy to help review and integrate it. (I don't remember anything about the implementation, so we would be on an equal footing :-) |
there are now several SQL query parsers in the community which we could rely on directly, in order to apply @gasche's plan. Although, to be really serious about this, I think one would need to evaluate all the other SQL type encoding that have emerged since then, notably Oleg's one. |
Camlp4 release for OCaml 4.08 is the last release ever - see the official announcement. |
I'm evaluating macaque for my own project which already moved to ppx. Unfortunately that means that macaque's camlp4 makes it incompatible. Are there any plans to port macaque to ppx?
The text was updated successfully, but these errors were encountered: