ocaml-systemd 1.3

[juergenhoetzel]

• Add compatibility with OCaml 5
• Build library using dune

[kit-ty-kate]

In the future it would be nice to replace this by a conf-libsystemd package to make it easier to maintain.

[kit-ty-kate]

#=== ERROR while compiling ocaml-systemd.1.3 ==================================#
# context              2.2.0~alpha~dev | linux/x86_64 | ocaml-base-compiler.4.02.3 | pinned(https://github.com/juergenhoetzel/ocaml-systemd/archive/1.3.tar.gz)
# path                 ~/.opam/4.02/.opam-switch/build/ocaml-systemd.1.3
# command              ~/.opam/opam-init/hooks/sandbox.sh build dune build -p ocaml-systemd -j 31
# exit-code            1
# env-file             ~/.opam/log/ocaml-systemd-7-f3820a.env
# output-file          ~/.opam/log/ocaml-systemd-7-f3820a.out
### output ###
# Error: I don't know about package ocaml-systemd (passed through
# --only-packages)

There is a mismatch between the name of the package and the name of the library. Is this expected?

Note that before my addition of -p name the package would have installed nothing at all.

[juergenhoetzel]

There is a mismatch between the name of the package and the name of the library. Is this expected?

No, i guess is should have omitted the prefix when submitting the first version of ocaml-systemd back in 2015.

I just hardcoded the systemd name in the amended commit.

[kit-ty-kate]

LGTM once CI finishes. Could you return the fixes in the opam file upstream to that they can be taken into account for next time?

Note also, another solution for future releases could also be to create a new systemd package and create a ocaml-systemd.transition package that points to it. Several packages made similar moves in the past and it avoids the extra annoying dune install (slower and more error prone than what opam does)

[kit-ty-kate]

Thanks