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

ocamldoc + -pack #4209

Open
vicuna opened this Issue Feb 16, 2007 · 3 comments

Comments

Projects
None yet
1 participant
@vicuna
Copy link
Collaborator

vicuna commented Feb 16, 2007

Original bug ID: 4209
Reporter: Julien Signoles
Status: acknowledged (set by @gasche on 2013-06-29T10:11:33Z)
Resolution: suspended
Priority: low
Severity: feature
Version: 3.09.2
Category: ocamldoc
Monitored by: dario Yoric

Bug description

$ mkdir a
$ touch a.ml a/a.ml
$ ocamlopt -c -I a -for-pack P a/a.ml
$ ocamlopt -c a.ml
$ ocamlopt -o p.cmx -pack -I a a/a.cmx
$ ocamldoc -I a a/a.ml a.ml
Fatal error: exception Failure("There are two implementations of module A.")

Additional information

Of course, without -pack one cannot have two compilation unit of the same name, and one can understand that ocamldoc doesn't handle that. But with -pack, the situation is different: we have two modules A and P.A, which is perfectly valid and works with ocamldc and ocamlopt, but not ocamldoc.

@vicuna

This comment has been minimized.

Copy link
Collaborator Author

vicuna commented Nov 12, 2007

Comment author: @zoggy

One would need to add a -pack option to ocamldoc, which it currently doesn't support. This would be rather heavy to implement, for an use case that seems quite exceptional. I'll mark that as "feature wish".

Il faudrait une option -pack à ocamldoc, ce qui n'est pas le cas et est assez lourd à implémenter, pour un usage plutôt exceptionnel il me semble.
Je le passe donc en souhait de fonctionnalité.

@vicuna

This comment has been minimized.

Copy link
Collaborator Author

vicuna commented Sep 23, 2008

Comment author: Yoric

Here is vote of support for such an option to be implemented.

@vicuna

This comment has been minimized.

Copy link
Collaborator Author

vicuna commented Jun 29, 2013

Comment author: @gasche

This feature has been requested by some other users since (eg. http://stackoverflow.com/questions/17368613/using-ocamldoc-with-packs in 2013, with mention of a similar discussion in 2011).

(Of course patches to implement the feature would be welcome.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.