gildor478 / ocaml-gettext

Other
13 stars 10 forks source link

Update the module path for Camomile >2.0.0 #27

Open dmbaturin opened 1 year ago

dmbaturin commented 1 year ago

Camomile 2.0.0 changed the module path — it moved CamomileLibraryDefault.Camomile to the root. I believe they should have kept the old path for compatibility (at least for a transitional period), but it's the way it is...

For now it's easier to make a trivial change to make packages build with new Camomile versions.

Note: Camomile 2.0.0 is only available for OCaml >=4.13, so this change will cause a de facto dependency requirement upgrade.

dmbaturin commented 1 year ago

For the record, AppVeyor failures have nothing to do with the PR — they are caused by Cygwin installation failures.

mbox note: Mirror Error:  Setup.ini signature https://cygwin.mirror.constant.com/x86_64/setup.xz.sig from https://cygwin.mirror.constant.com/ failed to verify.
Possible corrupt mirror?  Setup.ini rejected.

I created an issue in the CI script repository: ocaml/ocaml-ci-scripts/issues/359