-
Notifications
You must be signed in to change notification settings - Fork 89
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
META template and Melange #284
Comments
Related PR: #170 |
I think this template shouldn't be necessary anymore. It's only needed for pre ppxlib ppx's. If anyone is using bpf and nocrypto, those projects could easily be upgraded to modern ppx's. |
IIRC @jchavarri got an error when trying to remove the META file. @jchavarri could you paste the error you got when you get a chance? |
The error happens when trying to install packages that rely on ppx_deriving. these packages won't get the generated functions (like if the ppx was not applied): e.g.
|
Hi, I am trying to make ppx_deriving compatible with Melange. Unfortunately, Melange is not compatible with packages that have a META template, and Dune diligently makes that check when trying to build Melange packages.
When removing the META template, some packages that depend on it like
bpf
ornocrypto
fail to build because the PPX does not kick in, so the generated functions are not there.I wonder if there's a way to configure ppx_deriving in a way where the META template is not necessary, like other PPXs? So that Melange support can be added. I have read the Dune docs about META files but I'm still confused about why it's necessary in ppx_deriving case. Thanks!
cc @anmonteiro @rgrinberg
The text was updated successfully, but these errors were encountered: