karamel icon indicating copy to clipboard operation
karamel copied to clipboard

Switch to Dune buildsystem

Open XVilka opened this issue 6 years ago • 2 comments

Will improve modular building experience and speed. And pure ocamlbuild is being deprecated slowly in OCamlverse. It will make building on Windows platforms easier as well.

https://dune.readthedocs.io/en/latest/

XVilka avatar Jun 26 '19 15:06 XVilka

Hi XVilka,

Thanks for the bug report! A few points to keep in mind:

  • unless ocamlbuild breaks in a catastrophic manner, it's unlikely that I'll find cycles to upgrade the build system
  • kremlin is part of the larger everest project, and every change in tooling incurs debugging, upgrades, and general pain for the variety of users and setups that we have
  • we are committed to the mingw toolchain on windows, executed from a cygwin environment -- any switch to Dune would absolutely have to be compatible with that environment.

As such, I'm reluctant to undertake an effort now to switch to Dune, as it's likely to incur a lot of build complications. If you're willing to submit a pull request, I may be convinced to give it a go and try things out on CI and with other users? I would need your help when peoples' setups break and we need to debug what's going astray.

Thanks,

Jonathan

msprotz avatar Jun 26 '19 15:06 msprotz

I will check, it shouldn't be hard.

XVilka avatar Jun 26 '19 15:06 XVilka