From: Gabriel Scherer <gabriel.scherer@gmail.com>
To: Remco Vermeulen <r.vermeulen@vu.nl>
Cc: caml users <caml-list@inria.fr>
Subject: Re: [Caml-list] ocamldep & compilation units
Date: Sat, 3 Jan 2015 12:32:53 +0100 [thread overview]
Message-ID: <CAPFanBFHQFr4iQ66HefDP4-_yTRqfWAXr_Dn3GDYHZ9kZYy9oQ@mail.gmail.com> (raw)
In-Reply-To: <60E99B69-4F84-4C23-9FD1-3E773C1E7BA5@vu.nl>
[-- Attachment #1: Type: text/plain, Size: 3676 bytes --]
ocamldep is approximative by design (providing a correct list of
dependencies requires interleaving type-checking and dependency
computations, a very different *compiler* design that would require an
important amount of work). The easiest way out in the current state of
things is to have an option in build systems to remove spurious
dependencies by hand (they're quite rare in practice). ocamlbuild for
example has a (non_dependency "baz" "BAR") that you would use in your
situation. If Omake lacks such a capability, I think the best step forward
would be to add it (instead of trying to come up with different heuristics
than "ocamldep -modules").
On Sat, Jan 3, 2015 at 11:52 AM, Remco Vermeulen <r.vermeulen@vu.nl> wrote:
> >
> > On 02 Jan 2015, at 19:11, Gerd Stolpmann <info@gerd-stolpmann.de> wrote:
> >
> > Am Freitag, den 02.01.2015, 15:03 +0100 schrieb Remco Vermeulen:
> >> So my question is. is BAR in the above example correctly identified as
> a compilation unit by ocamldep?
> >
> > The syntax doesn't allow an unambiguous identification, so ocamldep
> > needs to take into account that BAR is a compilation unit. It doesn't
> > follow "open" when doing this, and I guess this is the point that
> > confuses you.
> What confused me is that the documentation says ocamldep -modules returns
> the module names of compilation units
> referenced in the source file, and then includes local modules when the
> “parent” module is implicitly qualified through open.
> When the local module is referenced explicitly (i.e, fully qualified), it
> is not included, which is inconsistent.
> >
> > The problem is that "ocamldep -modules" by definition can only analyze a
> > single module. The output is imprecise, however, and possible
> > inter-module effects are not taken into account (among other things). A
> > precise output would list BAR with the exception that it might be
> > shadowed by Foo.
> Perhaps a note should be added to the documentation of ocamldep about this.
> >
> > But imagine now we had the information with this degree of detail. As
> > omake wants to figure out the dependencies it would have to solve a
> > puzzle. In your case it is easily to solve, but in practice there are
> > often several "open" directives, and in this case you don't even know
> > whether "open Foo" opens a compilation unit. I am not sure whether a
> > well-performing algorithm even exists (did anybody tackle this
> > problem?).
> >
> > The workaround is to use naming schemes that allow you to clearly
> > distinguish between local modules and compilation unit (e.g. all your
> > local modules have 1-3 characters, and all compilation units have longer
> > names).
> This is not really an option, since this happens in an 3rd party library.
> It seems that patching the omake ocaml scanner, to not rely on the
> -modules option,
> seems to be the way to go as this is not trivial to handle in ocamldep.
>
> Thanks for your clarification!
>
> Cheers,
> Remco
>
> >
> > Gerd
> > --
> > ------------------------------------------------------------
> > Gerd Stolpmann, Darmstadt, Germany gerd@gerd-stolpmann.de
> > My OCaml site: http://www.camlcity.org
> > Contact details: http://www.camlcity.org/contact.html
> > Company homepage: http://www.gerd-stolpmann.de
> > ------------------------------------------------------------
>
> --
> Caml-list mailing list. Subscription management and archives:
> https://sympa.inria.fr/sympa/arc/caml-list
> Beginner's list: http://groups.yahoo.com/group/ocaml_beginners
> Bug reports: http://caml.inria.fr/bin/caml-bugs
>
[-- Attachment #2: Type: text/html, Size: 4909 bytes --]
next prev parent reply other threads:[~2015-01-03 11:33 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-02 14:03 Remco Vermeulen
2015-01-02 18:11 ` Gerd Stolpmann
2015-01-03 10:52 ` Remco Vermeulen
2015-01-03 11:32 ` Gabriel Scherer [this message]
2015-01-03 18:18 ` Xavier Leroy
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAPFanBFHQFr4iQ66HefDP4-_yTRqfWAXr_Dn3GDYHZ9kZYy9oQ@mail.gmail.com \
--to=gabriel.scherer@gmail.com \
--cc=caml-list@inria.fr \
--cc=r.vermeulen@vu.nl \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox