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

Closed
vicuna opened this issue Feb 16, 2007 · 4 comments
Closed

ocamldoc + -pack #4209

vicuna opened this issue Feb 16, 2007 · 4 comments

Comments

@vicuna
Copy link

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
Copy link
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
Copy link
Author

vicuna commented Sep 23, 2008

Comment author: Yoric

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

@vicuna
Copy link
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.)

@github-actions
Copy link

This issue has been open one year with no activity. Consequently, it is being marked with the "stale" label. What this means is that the issue will be automatically closed in 30 days unless more comments are added or the "stale" label is removed. Comments that provide new information on the issue are especially welcome: is it still reproducible? did it appear in other contexts? how critical is it? etc.

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