ocaml / ocamlbuild

The legacy OCamlbuild build manager
Other
121 stars 81 forks source link

Add support for OCaml 5.1 #319

Closed kit-ty-kate closed 1 year ago

kit-ty-kate commented 1 year ago

ocamlbuildlib and the ocamlbuild binary were wrongly compiled.

See https://github.com/dbuenzli/mtime/issues/45

gasche commented 1 year ago

Thanks!

The problem that you are fixing is that the Makefile is incorrect, it compiles all source files with -for-pack but only some of them are packed. (I guess that before this mistake was ignored by the compiler, and now it isn't anymore.)

Your fix is to create new source directories: only the files to be packed remain in src/, other library files go in plugin-lib/ and binaries go in bin/.

There are however two issues with the proposed change as-is, which should be relatively easy to fix:

kit-ty-kate commented 1 year ago

Done

gasche commented 1 year ago

Very nice, thanks!

gasche commented 1 year ago

(Do you have the necessary privileges to do a release, and would you be willing to do it? I hate releasing.)

kit-ty-kate commented 1 year ago

Done https://github.com/ocaml/opam-repository/pull/22182

gasche commented 1 year ago

You're the best.