ocsigen / ocsigen-start

Ocsigen-start: Higher-level library to develop Web and mobile applications with users, (pre)registration, notifications, etc.
Other
75 stars 32 forks source link

Get rid of camlp4 #370

Open rgrinberg opened 7 years ago

rgrinberg commented 7 years ago

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.

balat commented 7 years ago

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: