ocsigen-start icon indicating copy to clipboard operation
ocsigen-start copied to clipboard

Get rid of camlp4

Open rgrinberg opened this issue 9 years ago • 1 comments

I see that the starter project is currently using camlp4 for two syntax extensions - macaque and pgocaml.syntax. It would be great if this was avoided somehow.

Macaque should be ported to ppx. Although I suppose that if macaque hasn't been ported to ppx, it's really not getting that much usage. Perhaps ocsigen-start should reconsider using it?

As for pgocmal.syntax, I urge that its use should be reconsidered regardless of what progress it made towards ppx. The fact that pgocaml.syntax requires a project to have an active db connection just to compile and is only useful for static queries means that it doesn't justify it's usefulness to be a default for everyone.

rgrinberg avatar Dec 16 '16 01:12 rgrinberg

Hi Rudi,

Indeed there is no good solution for database interaction in OCaml right now.

Macaque is far from being usable for real scale project. It supports a very small subset of postgresql and uses a different syntax, which makes very difficult to write complex queries.

PGOCaml has the huge default you mention.

I suggest:

  • We switch to PGOCaml's PPX syntax when it's ready
  • Then we switch from macaque to PGOcaml in the OS lib

balat avatar Jan 06 '17 10:01 balat