fix(module_compilation): add dep to original source file #12774
Conversation
Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com>
anmonteiro
force-pushed
the
anmonteiro/source-preview-ppxed
branch
from
8114ac4 to
ba8200c
Compare
|
|
||
| let source_without_pp ~ml_kind t = | ||
| let source = | ||
| match (ml_kind : Ml_kind.t) with |
There was a problem hiding this comment.
I think this might need an adjustment for sources without mli's. That is, when ml_kind = Intf but there's not t.source.files.intf we should fall back to the .impl.
Would be great to have a test for this as well if you have the time.
There was a problem hiding this comment.
Thanks, will have a look.
|
@anmonteiro Can you say a few words what this change is needed? |
|
Here is the reproduction case: #12770 |
Thanks! |
|
@nojb any other questions or concerns about this change or its implementation that I should consider? |
| ; A "-c" | ||
| ; Command.Ml_kind.flag ml_kind | ||
| ; Dep src | ||
| ; Hidden_deps (Dep.Set.of_files (Option.to_list original)) |
There was a problem hiding this comment.
Would be nice to leave a comment why we need the source file here. It is in fact only needed for sandboxed builds as far as I can tell.
4 participants
No description provided.