-
Notifications
You must be signed in to change notification settings - Fork 412
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
Dune reports warnings twice #2452
Comments
The reason is that test.ml does not have a .mli which causes test.ml to be compiled twice; once with ocamlc (to produce the .cmi) and once with ocamlopt (to produce the .cmx) needed for test.exe. |
Closing as this behaviour is expected. |
That's unfortunate. Couldn't dune add |
This is a bit tedious, for instance if you only ask for the bytecode then we do need to display the warnings for the bytecode. Doing something different depending on what set of things are needed seems complicated to support and could hurt performance of incremental builds. |
The best is to always have a .mli file |
For example:
The text was updated successfully, but these errors were encountered: