Skip to content
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

Review install of mli and cmti files #7362

Closed
vicuna opened this issue Sep 19, 2016 · 3 comments
Closed

Review install of mli and cmti files #7362

vicuna opened this issue Sep 19, 2016 · 3 comments

Comments

@vicuna
Copy link

vicuna commented Sep 19, 2016

Original bug ID: 7362
Reporter: @dbuenzli
Status: closed (set by @damiendoligez on 2016-10-25T15:03:26Z)
Resolution: fixed
Priority: normal
Severity: minor
Version: 4.03.0
Target version: 4.04.0 +dev / +beta1 / +beta2
Fixed in version: 4.04.0 +dev / +beta1 / +beta2
Category: documentation
Monitored by: @gasche @hcarty @dbuenzli

Bug description

In the opkg project we automatically generate cross-linked API documentation for installed packages. The way packages are supposed to indicate what should be present in the generated API documentation is by installing mli and cmi (for ocamldoc generation) and cmti (for odoc generation) files for the modules that constitutes your API and you would like to see documented (more control through the package meta-data file may be provided in the future though).

While the ocamldoc path renders ocaml's package quite well with this convention (see http://docs.mirage.io/ocaml/index.html). It fails quite badly on the odoc path (see http://docs.mirage.io/odoc/ocaml/index.html).

There are two reasons.

  1. A lot of cmti files are installed for modules that are not part of the public API (at least their mli is not installed); those are mainly files in the compiler-libs/ directory.

  2. It seems that the cmti files for otherlibs are not installed, e.g. bigarray.cmti, str.cmti, unix.cmti, etc. are all missing.

@vicuna
Copy link
Author

vicuna commented Sep 19, 2016

Comment author: @dbuenzli

This issue b0-system/odig#1 could also maybe help.

@vicuna
Copy link
Author

vicuna commented Sep 27, 2016

Comment author: @dbuenzli

Partial patch in #827

@vicuna
Copy link
Author

vicuna commented Oct 25, 2016

Comment author: @damiendoligez

#827 was merged on 2016-10-21.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant