-
Notifications
You must be signed in to change notification settings - Fork 176
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
2.4.6: fails to compile on OSX with ocaml 4.02.1 #91
Comments
Indeed, the "ppx" library is empty: no ml mile, and an empty mli file (except for documentation). I'm not sure how to fix that. Maybe by remove this library and moving Ppx_lwt to the executable "ppx_lwt"? @Drup and @whitequark, what do you think? |
This was the first sane way I could think of to generate documentation for ppx, if anyone has a better idea .... |
Yes, we should definitely get rid of the empty library and properly append the META description using some other way. I have no idea what to do about documentation though. What about adding (Insert a rant about inflexibility of OASIS) |
Actually I think we could export the ppx rewriter itself as a library. This could be useful in construction of ppx drivers. This way, the corresponding .mli will get documented automatically. |
Any progress on fixing this on OSX? We have a dependency on trunk Lwt for the https://github.com/mirage/mirage-profile library, and this keeps coming up. The short term fix would just be to add a dummy ML file to the library to satisfy ar. |
See also ocaml-ppx/ppx_getenv#1 (comment) |
Empty libraries break the build on Mac OS X (see e.g. ocsigen/lwt#91). Not really sure what the purpose of test_frenetic was if it's empty, but this fixes the build...
Empty
ar
not allowed on OSXThe text was updated successfully, but these errors were encountered: