Implicit transitive deps experiment (do not merge)#14
Conversation
|
From that experiment all seems to work as expected with the new option except one part: commit 97b3d69 ("Why does this build?"). I am compiling cc @nojb (this does not relate to the new option, but is a question about |
I would expect so as well at first sight. Can you build at that commit and post the build log ( |
|
Ah @nojb I think I understand now, this is I believe thanks to https://github.com/ocaml/dune/blob/82bbef1c62e6f402e22823481c3bde35429a2bad/otherlibs/stdune/src/dune#L11 in Seems all fine then, I'm happy to consider this testing session satisfactory. |
Yes, this is the explanation. Thanks! |
|
Experimentation is complete - we'll go with #15 when ready. |
This further experiments with
(implicit_transitive_deps _)settings. Do not merge.