Conversation
|
Thanks! Getting up-to-date on standard dune idioms from more recent versions is a welcome change. Why is this marked as a draft? I am happy to merge this as long as the Dune builds keep more-or-less working. (On my machine I generally expect that |
|
I've marked this as a draft since I didn't have time to check it is currently working. It was working when I wrote it a few months back. I've undrafted it now. Please could you check that it doesn't break your existing workflow with dune. The new makefile target is to have finer control when building with dune after having built with make. It only removes the parts that confuses dune, which should soon be gone when we introduce the |
Signed-off-by: Ali Caglayan <alizter@gmail.com>
94bf6e9 to
995f160
Compare
|
I just tested it, and it works fine, thanks! I needed to rebase it on top of the current trunk to get the other dune fixes from #14328. (One of the fixes is also duplicated in the PR, but git seems able to rebase just fine.) Our |
|
Note: I tried to use |
Here are some improvements to the dune setup. The compiler isn't built using dune so I think its fine to ask for a more recent version here.