search

ocurrent / bun

tired of typing afl-fuzz? try bun!
MIT License
50 stars 10 forks source link

Make bun compatible with both Dune 1.11 and 2.x #15

Closed edwintorok closed 3 years ago

edwintorok commented 4 years ago

Otherwise trying to install bun would downgrade dune to 1.11.4, which would then fail to build my project which uses Dune 2.x.

See https://github.com/ocaml/dune/issues/2912, bun was relying on a dune feature that was part of a breaking change in 2.0.

Move the new target (creation of file) out of the alias into a separate rule, which then works on both old and new dune.

talex5 commented 3 years ago

Thanks - I've applied #18 which also adds support for dune 2.