Add another test for substituting in dependencies#1537
Add another test for substituting in dependencies#1537rgrinberg merged 3 commits intoocaml:masterfrom
Conversation
This raname was accidentally omitted Signed-off-by: Rudi Grinberg <rudi.grinberg@gmail.com>
Signed-off-by: Rudi Grinberg <rudi.grinberg@gmail.com>
|
I see thanks, it seems the problem was the error message I was getting because I was not using the right library name. I do wonder tho how the substitution happens for install files that are not part of a library, that is to say, data files in directory |
What mistake did you make? I'd like to improve dune to give proper error in such cases.
Yeah, indeed that's a bit confusing. I wonder if we should have a |
Use
Indeed thanks, we run into that case as the coq package has many files that are not part of a ML library; I've added a workaround for now by declaring a dummy lib, however it may not be ideal; not sure tho. |
|
What would On the other hand, we could have |
That'd look great indeed; and would surely suffice for most cases. For Coq's object files we may end up using |
What I had in mind was |
|
We could also have: |
|
I'm probably overestimating the risks here, but it's possible that a new section would be added that could conflict with an existing form. |
cc @ejgallego it seems to work here.