Support new locations of unix, str and dynlink#5581
Conversation
rgrinberg
left a comment
There was a problem hiding this comment.
Needs a CHANGES entry for 2.9.4
|
It does also need the upstream PR in OCaml to be merged first! |
|
Rebased and added Changes entry to 2.9.4 (is it correct that it goes in the 2.9.4 section with PR number for main?). The OCaml PR for this is about to be merged. |
Signed-off-by: David Allsopp <david.allsopp@metastack.com>
|
The flambda-backend compiler is planning to integrate ocaml/ocaml#11198 on a 4.14 branch, so we'd like some support for that in dune too. I've got a patch which more or less lowers the version bound for the code of this PR to 4.14, which should work because using the new code on a vanilla 4.14 compiler will still produce the expected result. |
|
Presumably it also has to check for the files in two locations for the |
|
Could you point me to the |
Updates Dune for compatibility with the changes proposed in ocaml/ocaml#11198.