Marcello Seri

Results 435 comments of Marcello Seri

Closing since this was merged

Should I mark it as do-not-merge then?

It doesn't look like there is any failure due to this package honestly, besides maybe windows (but I am re-running it, the previous failure was from the CI)

I have merged since there seems to be nothing to do here

We could create a new conf-libssl version with the correct package name, however centos7 is no longer supported so we cannot test it. Otherwise it is currently possible to work...

Can you drop the dependency on `bytes`: see https://github.com/ocaml/dune/issues/650 It can fail as ``` #=== ERROR while compiling archive.3.7.2+5 ====================================# # context 2.4.0~alpha1~dev | linux/x86_64 | ocaml-base-compiler.5.0.0 | pinned(https://github.com/gildor478/ocaml-archive/releases/download/v3.7.2%2B5/archive-3.7.2.5.tbz) #...

This looks ok at least as a temporary solution and it is more precise than hardcoding the path as done for gmp, but it is still not working on macos...

@hcarty I cannot promise very active maintainership, but indeed I still use the library, so I would be happy to at least make these small fixes and release them

This has now been released, and seems to work on all the opam-repository CI targets. Thanks @conroj

Thanks for looking into it so promptly 🙏